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

AGM Location Change PDF file name - "AGM" should be in all caps #18469

Closed
yuisotozaki opened this issue Nov 3, 2023 · 6 comments
Closed

AGM Location Change PDF file name - "AGM" should be in all caps #18469

yuisotozaki opened this issue Nov 3, 2023 · 6 comments
Assignees
Labels
bug Something isn't working ENTITY Business Team Priority2 UX Assurance

Comments

@yuisotozaki
Copy link

AGM Location Change PDF file name - "AGM" should be in all caps

Private Zenhub Image

@yuisotozaki yuisotozaki added bug Something isn't working ENTITY Business Team UX Assurance labels Nov 3, 2023
@severinbeauvais
Copy link
Collaborator

This issue will also affect the Letter Of AGM Extension Change.

@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Nov 9, 2023

The root cause of this issue is that the UI generates the name from the document's key, ie:

Private Zenhub Image

In this example, letterOfAgmLocationChange becomes "Letter Of Agm Location Change".

To solve this will require the Legal API to pass the "name" as well as (or instead of) the key value. Note that this may affect third party users. Also, Filings UI will need to be updated to accept the new API response.

@jdyck-fw
Copy link
Collaborator

Sev may have an easy answer on this, and 1 point is to explore if it is so

@severinbeauvais
Copy link
Collaborator

In my opinion, the cost of doing this "ideally" is too high for this "simple" fix, so I implemented a special-case solution that capitalizes "Agm" -> "AGM".

Special cases are not ideal but in this case it's simple, extensible, understandable and safe, so I think it's OK.

If we get a lot of special cases then we'll want to look at a better solution, but we're not there yet.

@severinbeauvais
Copy link
Collaborator

Test Notes

Look at the title of the output letter for an AGM Location Change (or AGM Extension).

@yuisotozaki
Copy link
Author

Ready for QA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ENTITY Business Team Priority2 UX Assurance
Projects
None yet
Development

No branches or pull requests

4 participants