-
Notifications
You must be signed in to change notification settings - Fork 10
feat(structures): making sms day mandatory #3852
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR implements mandatory SMS day selection by adding validation and UI changes. The implementation correctly adds both client-side validation and user feedback, but has a critical typo in the error message that needs fixing. The validation logic is sound and properly handles both the "at least one day" and "maximum two days" requirements.
...d/src/app/modules/structures/components/structures-sms-form/structures-sms-form.component.ts
Outdated
Show resolved
Hide resolved
...d/src/app/modules/structures/components/structures-sms-form/structures-sms-form.component.ts
Outdated
Show resolved
Hide resolved
3fe0867
to
572c2a6
Compare
…ures-sms-form/structures-sms-form.component.ts Co-authored-by: Revu <dnum.fabrique@sg.social.gouv.fr>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3852 +/- ##
==========================================
- Coverage 65.78% 65.69% -0.10%
==========================================
Files 970 971 +1
Lines 16220 16249 +29
Branches 2237 2251 +14
==========================================
+ Hits 10671 10674 +3
- Misses 5208 5528 +320
+ Partials 341 47 -294
... and 129 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An error occurred: Error processing line comments: [
{
"expected": "array",
"code": "invalid_type",
"path": [
"comments"
],
"message": "Invalid input: expected array, received string"
}
]
1ca7311
to
6ba95bb
Compare
packages/backend/src/_migrations/1755033531923-manual-migration.ts
Outdated
Show resolved
Hide resolved
packages/backend/src/database/entities/structure/StructureTable.typeorm.ts
Show resolved
Hide resolved
...d/src/app/modules/structures/components/structures-sms-form/structures-sms-form.component.ts
Show resolved
Hide resolved
...src/app/modules/structures/components/structures-sms-form/structures-sms-form.component.html
Outdated
Show resolved
Hide resolved
794ff95
to
6679f04
Compare
|
🎉 This PR is included in version 2.215.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
No description provided.