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

ENP SQS Send Message Pre-Process Asynchronously #154

Open
5 tasks
EvanParish opened this issue Jan 31, 2025 · 0 comments
Open
5 tasks

ENP SQS Send Message Pre-Process Asynchronously #154

EvanParish opened this issue Jan 31, 2025 · 0 comments
Labels
Dev Reviewed Reviewed by Tech Lead Notify Board trigger PM Reviewed Reviewed by Product Manager QA Reviewed Reviewed by Quality Assurance QA Issue requires QA collaboration

Comments

@EvanParish
Copy link
Contributor

EvanParish commented Jan 31, 2025

User Story - Business Need

When we receive an identifier to our POST notification endpoint we want to use all the previously identified Celery tasks necessary and chain them together using aiobotocore. This combined with the deliver_sms work will allow 100% of SMS requests to bypass the notification-api ECS service.

  • Sync with Kyle when ticket is picked up.
  • Ticket is understood, and QA has been contacted (if the ticket has a QA label).

User Story(ies)

As a backend engineer
I want to prepare an envelope for the identified Celery tasks from the API
So that we can work on implementing the sms route in the ENP repo

Additional Info and Resources

  • aiobotocore repo
  • Previous work identified which Celery tasks needed to be chained, so this will correctly chain them.

Acceptance Criteria

  • lookup_va_profile_id, lookup_contact_info, and deliver_sms chain works as expected
  • lookup_contact_info, and deliver_sms chain works as expected
  • This work is added to the sprint review slide deck (key win bullet point and demo slide)

QA Considerations

This should result in the same output as the notification-api.

Potential Dependencies

Determine Celery tasks necessary based on incoming request.

Out of Scope

Replacing any functionality of the Celery tasks.

@EvanParish EvanParish added the Notify Board trigger label Jan 31, 2025
@cris-oddball cris-oddball added the QA Reviewed Reviewed by Quality Assurance label Feb 3, 2025
@k-macmillan k-macmillan added QA Issue requires QA collaboration Dev Reviewed Reviewed by Tech Lead labels Feb 3, 2025
@kbelikova-oddball kbelikova-oddball added the PM Reviewed Reviewed by Product Manager label Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev Reviewed Reviewed by Tech Lead Notify Board trigger PM Reviewed Reviewed by Product Manager QA Reviewed Reviewed by Quality Assurance QA Issue requires QA collaboration
Projects
None yet
Development

No branches or pull requests

5 participants