Skip to content
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

Remove feature flag VA_PROFILE_SMS_STATUS_ENABLED #2162

Open
6 tasks
MackHalliday opened this issue Dec 5, 2024 · 0 comments
Open
6 tasks

Remove feature flag VA_PROFILE_SMS_STATUS_ENABLED #2162

MackHalliday opened this issue Dec 5, 2024 · 0 comments

Comments

@MackHalliday
Copy link

MackHalliday commented Dec 5, 2024

User Story - Business Need

We need to remove the VA_PROFILE_SMS_STATUS_ENABLED feature flag once sufficient time has passed and feature works as expected.

  • Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)

As a VA Notify developer
I want remove the VA_PROFILE_SMS_STATUS_ENABLED feature flag
So that code is clean and continues to be maintainable.

Additional Info and Resources

Acceptance Criteria

  • Feature flag is removed VA_PROFILE_SMS_STATUS_ENABLED from code path, task definitions, and tests
  • SMS notifications statuses continue to send to VA Profile without error
  • Any related tests are updated or removed
  • This work is added to the sprint review slide deck (key win bullet point and demo slide)

QA Considerations

  • Validate that sms and email notifications are sent to VA Profile without error.

Potential Dependencies

Need to be okayed by VA Profile - Ksenia/Kyle with coordinate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants