PLASMA-8275: add custom quarterNames in Calendar-like - #3198
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (65)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughCalendar, DatePicker, DatePickerRange, and DateTimePicker now accept custom names for the four quarters. The change passes those names to quarter views and date-info logic, retains default names when entries are absent, and adds tests and documentation. ChangesCustom quarter labels in Calendar
Priority: ➖ Normal Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant Caller
participant DatePicker
participant useDatePicker
participant Calendar
Caller->>DatePicker: quarterNames prop
DatePicker->>useDatePicker: quarterNames option
DatePicker->>Calendar: quarterNames prop
Calendar-->>Caller: quarter labels in calendar view
useDatePicker-->>Caller: quarter name in date info
Merge Risk: ⚪ Minimal · up to Custom quarter labels appear ready to merge after normal checks; no actionable merge-blocking issue is established. Security Architecture ReviewSecurity architecture risk: 🔵 Low · up to Custom labels change what applications display and receive in date callbacks. The inspected components retain their date-selection checks, and no new privileged operation was identified. Risk remains low rather than minimal because downstream applications may handle callback labels in ways that could not be assessed here. Retained concerns Security review detailsSecurity Blast Radius
Trust Boundaries and Controls
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
|
Theme Builder app deployed! https://plasma.sberdevices.ru/pr/plasma-theme-builder-pr-3198/ |
|
Ок |
9249636 to
8c4f82f
Compare
Core
Calendar, Datepicker, DateTimePicker
What/why changed
Summary by CodeRabbit
📦 Published PR as canary version:
Canary Versions✨ Test out this PR locally via:
npm install @salutejs/plasma-asdk@0.396.0-canary.3198.36391526963.0 npm install @salutejs/plasma-b2c@1.638.0-canary.3198.36391526963.0 npm install @salutejs/plasma-colors@0.24.0-canary.3198.36391526963.0 npm install @salutejs/plasma-core@1.245.0-canary.3198.36391526963.0 npm install @salutejs/plasma-giga@0.365.0-canary.3198.36391526963.0 npm install @salutejs/plasma-homeds@0.365.0-canary.3198.36391526963.0 npm install @salutejs/plasma-hope@1.392.0-canary.3198.36391526963.0 npm install @salutejs/plasma-icons@1.251.0-canary.3198.36391526963.0 npm install @salutejs/plasma-new-hope@0.382.0-canary.3198.36391526963.0 npm install @salutejs/plasma-tokens@1.156.0-canary.3198.36391526963.0 npm install @salutejs/plasma-tokens-b2b@1.67.0-canary.3198.36391526963.0 npm install @salutejs/plasma-tokens-b2c@0.78.0-canary.3198.36391526963.0 npm install @salutejs/plasma-tokens-core@0.15.0-canary.3198.36391526963.0 npm install @salutejs/plasma-tokens-web@1.82.0-canary.3198.36391526963.0 npm install @salutejs/plasma-typo@0.55.0-canary.3198.36391526963.0 npm install @salutejs/plasma-web@1.640.0-canary.3198.36391526963.0 npm install @salutejs/sdds-bizcom@0.370.0-canary.3198.36391526963.0 npm install @salutejs/sdds-cs@0.374.0-canary.3198.36391526963.0 npm install @salutejs/sdds-dfa@0.368.0-canary.3198.36391526963.0 npm install @salutejs/sdds-finai@0.361.0-canary.3198.36391526963.0 npm install @salutejs/sdds-icons@0.9.0-canary.3198.36391526963.0 npm install @salutejs/sdds-insol@0.365.0-canary.3198.36391526963.0 npm install @salutejs/sdds-insol-next@0.364.0-canary.3198.36391526963.0 npm install @salutejs/sdds-netology@0.369.0-canary.3198.36391526963.0 npm install @salutejs/sdds-os@0.40.0-canary.3198.36391526963.0 npm install @salutejs/sdds-platform-ai@0.369.0-canary.3198.36391526963.0 npm install @salutejs/sdds-sbcom@0.370.0-canary.3198.36391526963.0 npm install @salutejs/sdds-scan@0.368.0-canary.3198.36391526963.0 npm install @salutejs/sdds-serv@0.369.0-canary.3198.36391526963.0 npm install @salutejs/sdds-vc@0.4.0-canary.3198.36391526963.0 npm install @salutejs/core-themes@0.43.0-canary.3198.36391526963.0 npm install @salutejs/plasma-themes@0.65.0-canary.3198.36391526963.0 npm install @salutejs/sdds-themes@0.83.0-canary.3198.36391526963.0 npm install @salutejs/sdds-api-tests@0.27.0-canary.3198.36391526963.0 npm install @salutejs/plasma-cy-utils@0.175.0-canary.3198.36391526963.0 npm install @salutejs/plasma-sb-utils@0.246.0-canary.3198.36391526963.0 npm install @salutejs/plasma-tokens-utils@0.63.0-canary.3198.36391526963.0 # or yarn add @salutejs/plasma-asdk@0.396.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-b2c@1.638.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-colors@0.24.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-core@1.245.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-giga@0.365.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-homeds@0.365.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-hope@1.392.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-icons@1.251.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-new-hope@0.382.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-tokens@1.156.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-tokens-b2b@1.67.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-tokens-b2c@0.78.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-tokens-core@0.15.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-tokens-web@1.82.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-typo@0.55.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-web@1.640.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-bizcom@0.370.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-cs@0.374.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-dfa@0.368.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-finai@0.361.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-icons@0.9.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-insol@0.365.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-insol-next@0.364.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-netology@0.369.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-os@0.40.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-platform-ai@0.369.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-sbcom@0.370.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-scan@0.368.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-serv@0.369.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-vc@0.4.0-canary.3198.36391526963.0 yarn add @salutejs/core-themes@0.43.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-themes@0.65.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-themes@0.83.0-canary.3198.36391526963.0 yarn add @salutejs/sdds-api-tests@0.27.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-cy-utils@0.175.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-sb-utils@0.246.0-canary.3198.36391526963.0 yarn add @salutejs/plasma-tokens-utils@0.63.0-canary.3198.36391526963.0