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

Production release 20231120 #796

Merged
merged 4 commits into from
Nov 20, 2023
Merged

Production release 20231120 #796

merged 4 commits into from
Nov 20, 2023

Conversation

jkeasley
Copy link
Collaborator

This release deploys the change that fixes the build process to production,
along with a couple of minor branding changes:

  • changing the logo to the NHS England logo, ensuring that it displays
    at 60px high, and changing the Site name text from
  • "NHS England - Transformation Directorate" to "Transformation Directorate"

To test:

  • view any page on the local site and confirm that the logo now displays
    NHS England, rather than just NHS
  • Confirm that the Site title displayed is now Transformation Directorate.

The build of the nhsx site was being broken by some incompatible changes
which had been made to poetry.

These changes enforce the use of poetry 1.2.2 which works with the
pyproject file as currently configured, and should allow the build
to complete allowing the site to be deployed again.

to test:
download this branch, and uncomment the line 127 in docker/web/Dockerfile
then run script/setup to build the site. confirm that the build completes.

this should confirm that the build as configured in Dockerfile-prod
will work as expected.
This commit makes some minor changes to the branding of the NHSx site
changing the logo to the NHS England logo, ensuring that it displays
at 60px high, and changing the Site name text from
"NHS England - Transformation Directorate" to "Transformation Directorate"

To test:
- view any page on the local site and confirm that the logo now displays
NHS England, rather than just NHS
- Confirm that the Site title displayed is now Transformation Directorate.

You may need to clear the cache in the admin section to see the changes
@jkeasley jkeasley merged commit 9b97d12 into master Nov 20, 2023
1 check passed
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