department-of-veterans-affairs / notification-api

Notification API
MIT License
16 stars 8 forks source link

Remove flask-cors Dependency #1794

Closed k-macmillan closed 2 months ago

k-macmillan commented 4 months ago

User Story - Business Need

Flask-Cors was necessary for the old UI, but is no longer necessary with the SSUI. Remove it and all associated code.

User Story(ies)

As a VA Notify Engineer I want to keep our system lean and free of unused dependencies So that we have less to maintain and fewer vulnerabilities

Additional Info and Resources

Acceptance Criteria

mjones-oddball commented 4 months ago

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @k-macmillan @kalbfled @ldraney @MackHalliday @mchlwellman

EvanParish commented 2 months ago

The code is complete and unit tests pass locally. I'll deploy and test on Dev in the morning.