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

Organisation description fix #9648

Merged
merged 2 commits into from
Nov 25, 2024
Merged

Organisation description fix #9648

merged 2 commits into from
Nov 25, 2024

Conversation

minhngocd
Copy link
Contributor

ZenDesk enquiry to fix up some copy of Organisation descriptions.

https://trello.com/c/oLQQId8c/3195-technical-fault-with-whitehall

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

When there are organisations with a parent and also multiple children, we need to make the description grammatically correct. At the moment it says "org 1 works with org 2 is supported by 3....". Appending "and" to make the sentence read better.
Copy link
Contributor

@ryanb-gds ryanb-gds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, though it feels real weird to me that this text is being constructed by Whitehall rather than being built by the frontend app from the content item links

@minhngocd
Copy link
Contributor Author

LGTM, though it feels real weird to me that this text is being constructed by Whitehall rather than being built by the frontend app from the content item links

+1 to that - maybe we stick something in tech debt... I was also expecting to put the fix into the rendering app but when traced back, it was attached to the content item all along...

@minhngocd minhngocd merged commit bad67db into main Nov 25, 2024
19 checks passed
@minhngocd minhngocd deleted the organisation-description-fix branch November 25, 2024 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants