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

Release #3024

Merged
merged 3 commits into from
Apr 18, 2024
Merged

Release #3024

merged 3 commits into from
Apr 18, 2024

Conversation

boxadesign
Copy link
Contributor

(Please add a sentence or two explaining the context of this PR. Reviewers can go to the ticket for detail.)
CONTEXT: This changeset adds/removes/updates [summary of feature/area/code] so that [summary of benefit].

Tick or delete as appropriate:

Workflow

Reviewing help

  • Explains how to test locally, including how to set up appropriate data
  • Includes screenshot(s) - ideally before and after, but at least after

Housekeeping

  • Added all new environment variables to Vault.
  • Cleaned up old feature flags
  • Upgraded any vulnerable dependencies.
  • Ran the make manage download_geolocation_data command
  • I have updated security dependencies
  • Python requirements have been re-compiled.
  • Frontend assets have been re-compiled.
  • I have checked that my PR is using the latest package versions of: great-components, directory-constants, directory-healthcheck, directory-validators, directory-components, directory-api-client, directory-ch-client, django-staff-sso-client, directory-forms-api-client, directory-sso-api-client, sigauth

Merging

  • This PR can be merged by reviewers. (If unticked, please leave for the author to merge)

boxadesign and others added 3 commits April 17, 2024 10:33
* Update styling to cookie modal

* Updates to styling for button

* revert button styling

* Fix overflow on modal

* More styling updates and cookie confirmation wip

* Fix syntax error in cookie notice string

* More styling and accessibility improvements to the confirmation banner

* recompile styles

* Update ui tests

* Reinstall npm packages and rebuild assets

* Remove cookie notice as test

* Remove cookie notice content as test

* Remove translate function from cookie notice

* Remove cookie notice content as test

* Remove cookie notice content as test

* Update preferences link code

* Update preferences link code

* Cleaned up styles from demo

* Adjust robots.txt to see if it makes a difference to lighthouse

* Attempt to debug lighthouse

* reset lighthouse debugging

* change lighthouse debugging

* add defer to some js script references

* Run the webserver as a step for lighthouse

* revert and reduce score threshold for performance

---------

Co-authored-by: Bobby Stearman <[email protected]>
GREATUK-597: Update performance dashboard banner wording
@boxadesign boxadesign requested a review from a team as a code owner April 18, 2024 10:23
@boxadesign boxadesign merged commit 85ef376 into master Apr 18, 2024
7 checks passed
stuart-mindt pushed a commit that referenced this pull request Jun 6, 2024
* GREATUK 40 - cookie modal accessibility updates (#3014)

* Update styling to cookie modal

* Updates to styling for button

* revert button styling

* Fix overflow on modal

* More styling updates and cookie confirmation wip

* Fix syntax error in cookie notice string

* More styling and accessibility improvements to the confirmation banner

* recompile styles

* Update ui tests

* Reinstall npm packages and rebuild assets

* Remove cookie notice as test

* Remove cookie notice content as test

* Remove translate function from cookie notice

* Remove cookie notice content as test

* Remove cookie notice content as test

* Update preferences link code

* Update preferences link code

* Cleaned up styles from demo

* Adjust robots.txt to see if it makes a difference to lighthouse

* Attempt to debug lighthouse

* reset lighthouse debugging

* change lighthouse debugging

* add defer to some js script references

* Run the webserver as a step for lighthouse

* revert and reduce score threshold for performance

---------

Co-authored-by: Bobby Stearman <[email protected]>

* GREATUK-597: Update performance dashboard banner wording

---------

Co-authored-by: Bobby Stearman <[email protected]>
Co-authored-by: divyaparameswaran <[email protected]>
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