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.

Sign up for GitHub

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

Jump to bottom

Generate multilingual screenshots for wp.org with playwright #783

Merged
merged 28 commits into from
Aug 14, 2024
Merged

Generate multilingual screenshots for wp.org with playwright #783

merged 28 commits into from
Aug 14, 2024

Conversation

carstingaxion
Copy link
Collaborator

@carstingaxion carstingaxion commented Aug 14, 2024 •
edited
Loading

Description of the Change

Generating screenshots in multiple languages for the plugin and keeping them up to date with the development might become a time intensive task.
#766

This PR automates this process, starting with 4 screenshots to generate in 7 locales.

Closes #766

How to test the Change

Because of the needed permissions for the workflow to run, it needs to be merged first to be run-able.

After merging, this PR creates a new manually action to trigger the screenshot generation.

Changelog Entry

Added - Generate multilingual screenshots for wp.org with playwright

Credits

Props @carstingaxion

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@carstingaxion
Add screenshot-generator scripts
a3d785f
@carstingaxion
Add wordpress/e2e-test-utils-playwright as dev-dep
281045b
@carstingaxion
Add wordpress/env as dev-dep
2346090
@carstingaxion
Add screenshot-generator scripts (2/2)
731a329
@carstingaxion
Added 2 more screenshots (to take), run in localised envs, renamed fi…
e181c42 
…les & folders
@carstingaxion
Run only manually
8a33e60
@carstingaxion
Rename workflow to align with other workflow names
f7094f2
@carstingaxion
TESTING // REVERT BEFORE MERGE
519066e
@carstingaxion
Fix 'Invalid workflow file:' error
03fcdd4
@carstingaxion
Fix 'Invalid workflow file:' error
31457e6
@carstingaxion
Prevent parallel runs to make use of the caching for node_modules and…
bb5e3c8 
… playwright browsers
@carstingaxion
Fix 'wp-env: command not found' error
22a99e4
@carstingaxion
Run commit & PR creation on failure()
83e2ead
@carstingaxion
Ignore artifacts folder that is used by wp-env
04c0044
@carstingaxion
Create a branch before creating a PR
b89d4f6
@carstingaxion
Set workflow permissions to allow to create a PR
23b29a4
@carstingaxion
Push changes to origin before PR
3db0e27
@carstingaxion
Disable welcome guide inside post editor
87107d2
@carstingaxion
Set 'continue-on-error' for first run of screenshot generation
5cd3ceb
@carstingaxion
Re-Enable cancelation of concurrent workflow runs
cd1c33b
@carstingaxion
Install all browsers
2a747ea
@carstingaxion
Do not use failure() with continue-on-error
148a610
@carstingaxion
Grant write permissions for the contents scope to allow pushing changes
8d8bce9
@carstingaxion
Install languages in wp-env
6c929ab
@carstingaxion
Use all locales with more than 90% complete translations on https://t…
37bec45 
…ranslate.wordpress.org/projects/wp-plugins/gatherpress/
@carstingaxion
Remove pull_request trigger and clean indention
a9f0ea8
@carstingaxion
Re-Order Screenshot & update README
7caf980
@carstingaxion carstingaxion changed the title Feature/generate screenshots w playwright Generate screenshots for wp.org with playwright Aug 14, 2024
@carstingaxion carstingaxion marked this pull request as ready for review August 14, 2024 14:26
@carstingaxion carstingaxion changed the title Generate screenshots for wp.org with playwright Generate multilingual screenshots for wp.org with playwright Aug 14, 2024
@mauteri mauteri merged commit 7a11b64 into GatherPress:main Aug 14, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mauteri mauteri mauteri approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Generate multilingual screenshots automatically for each release
2 participants
@carstingaxion @mauteri