Releases/published official releases/official release 2025.4#4354
Conversation
1- fixed loading of runset config having new Ginger Play configs 2- fixing Ginger Play pop up which hide behind app 3- improving Ginger Play icon style 4- aligning Ginger play icon in diffrent pages
…ases/Official-Release-2025.4' into Fix-2025.4-release-bugs
Fix 2025.4 release bugs
|
Caution Review failedThe pull request is closed. WalkthroughThis PR enhances the Ginger Play feature integration by redesigning the UI components, introducing an enablement check, and refactoring deserialization logic. Changes include a revamped Ginger Play button with layered styling, updated image assets, a new utility method for checking Ginger Play status, and modifications to Learn wizard labeling and configuration loading. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant TwoLevelMenuPage
participant GingerPlayUtils
participant Repository
participant BannerWindow
User->>TwoLevelMenuPage: Click Pro Feature Button
TwoLevelMenuPage->>GingerPlayUtils: IsGingerPlayEnabled()
GingerPlayUtils->>Repository: Read GingerPlayConfiguration
alt Configuration exists
Repository-->>GingerPlayUtils: GingerPlayConfiguration
GingerPlayUtils-->>TwoLevelMenuPage: true/false (enablement status)
else Configuration missing/Error
Repository-->>GingerPlayUtils: null/Exception
GingerPlayUtils-->>TwoLevelMenuPage: false
end
alt GingerPlay enabled & Gateway URL valid
TwoLevelMenuPage->>BannerWindow: Create & Show
TwoLevelMenuPage->>BannerWindow: Set Owner to Main Window
BannerWindow-->>User: Display Banner
else Not enabled or missing URL
TwoLevelMenuPage-->>User: Feature unavailable
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: ASSERTIVE Plan: Pro 📒 Files selected for processing (6)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
Type of Change
Checklist
[IsSerializedForLocalRepository]where neededReporter.ToLog()patternSummary by CodeRabbit
New Features
Style