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

Added more Optimo article URLs for testing #9720

Merged
merged 3 commits into from
Dec 3, 2021
Merged

Conversation

LilyL0u
Copy link
Contributor

@LilyL0u LilyL0u commented Dec 2, 2021

As Optimo is being used more now and more things are being added to these pages I wanted to add some more URLs and a smoke test to improve coverage.

I will add more when I add tests for a specific feature, to ensure that page covers that feature.


  • I have assigned myself to this PR and the corresponding issues
  • I have added the cross-team label to this PR if it requires visibility across World Service teams
  • I have assigned this PR to the Simorgh project
  • (BBC contributors only) This PR follows the repository use guidelines

Testing:
All pass on local, test and live

@LilyL0u LilyL0u added cypress Cypress changes are required cross-team For visibility for both World Service teams (Engage & Media) optimo labels Dec 2, 2021
@LilyL0u LilyL0u self-assigned this Dec 2, 2021
@LilyL0u LilyL0u changed the title Added more optimo articles for testing Added more Optimo article URLs for testing Dec 2, 2021
@LilyL0u LilyL0u added this to Code Review in Simorgh Dec 3, 2021
@LilyL0u LilyL0u merged commit e10856c into latest Dec 3, 2021
@LilyL0u LilyL0u deleted the add-more-optimo-urls branch December 3, 2021 16:43
@pvaliani pvaliani moved this from Code Review to Done in Simorgh Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team For visibility for both World Service teams (Engage & Media) cypress Cypress changes are required optimo
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants