fecgov / fecfile-web-app

Other
6 stars 4 forks source link

Web-App: Missing `/` in API calls causing unnecessary redirect #2366

Open exalate-issue-sync[bot] opened 1 week ago

exalate-issue-sync[bot] commented 1 week ago

Add / to API calls that are missing them so that we avoid unnecessary redirects.

Recreating the issue:

![image (1).png](/attachments/13517?name=image (1).png)

QA Notes

DEV Notes

Build out unit tests that future-proof so we don’t run into this error again.

For testing, provide proof of unit tests and E2E tests passing

Design

null

See full ticket and images here: FECFILE-1785

exalate-issue-sync[bot] commented 1 week ago

akhorsand commented: [~accountid:712020:2a1493e5-adee-45bd-b27e-868a5c8d3f62] what should the outcome of this ticket be? This is assigned for Sprint 52 - can you add some detail and get it ready for pointing?

exalate-issue-sync[bot] commented 6 days ago

Todd Lees commented: [~accountid:61b0b42cd5986c006a9e1c94] added prescriptive content

exalate-issue-sync[bot] commented 5 days ago

akhorsand commented: Moving back to AA to talk through potential unit tests.

exalate-issue-sync[bot] commented 5 days ago

Sasha Dresden commented: Created a PR to demo using an interceptor with some generic tests for it. [https://github.com/fecgov/fecfile-web-app/pull/2390|https://github.com/fecgov/fecfile-web-app/pull/2390|smart-link]