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

Use the same metadata for all pages #516

Merged
merged 1 commit into from Nov 3, 2017
Merged

Use the same metadata for all pages #516

merged 1 commit into from Nov 3, 2017

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Nov 3, 2017

  • Don’t hide the part of bits of metadata on new taxonomy pages
  • We will be iterating what nav is shown in future multivariate tests
  • Part of includes both trivially related things and important related
    things – we need to do better in distinguishing them

Review app vs live:
https://government-frontend-pr-516.herokuapp.com/government/publications/national-apprenticeship-awards-apprentice-categories
https://www.gov.uk/government/publications/national-apprenticeship-awards-apprentice-categories?ABTest-EducationNavigation=B

https://trello.com/c/zuBOPznV/74-remove-ab-test-from-government-frontend

@boffbowsh boffbowsh temporarily deployed to government-frontend-pr-516 Nov 3, 2017 Inactive
* Don’t hide the `part of` bits of metadata on new taxonomy pages
* We will be iterating what nav is shown in future multivariate tests
* Part of includes both trivially related things and important related
things – we need to do better in distinguishing them
@fofr fofr merged commit 0e27c2f into master Nov 3, 2017
2 checks passed
2 checks passed
continuous-integration/jenkins/branch This commit looks good
Details
continuous-integration/jenkins/publishing-e2e-tests Publishing end-to-end tests succeeded on Jenkins
Details
@fofr fofr deleted the no-custom-metadata branch Nov 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.