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

WP-NOW: update documentation adding quickstart section and updating GIF #205

Merged
merged 4 commits into from Mar 29, 2024

Conversation

sejas
Copy link
Collaborator

@sejas sejas commented Mar 27, 2024

What?

  • Update wp-now Readme for better simplicity and highlighting npx usage.

Why?

Testing Instructions

Confirm the README looks good.

https://github.com/WordPress/playground-tools/blob/aac36a5676ba9d5da1d2d44ca1590337017c3ac8/packages/wp-now/README.md

wp-now-quickstart

@sejas sejas self-assigned this Mar 27, 2024
Copy link
Contributor

@katinthehatsite katinthehatsite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good on my end.

@StevenDufresne
Copy link
Contributor

StevenDufresne commented Mar 28, 2024

Nice!

Can we delay the last frame in the GIF for an extra 2 seconds where we show the server URL since it's the aha moment and the handoff for our users?

@StevenDufresne
Copy link
Contributor

Readme changes look great!

@adamziel
Copy link
Collaborator

Lovely!

@sejas sejas merged commit 7d6815c into trunk Mar 29, 2024
2 checks passed
@sejas sejas deleted the update/wp-now-documentation branch March 29, 2024 09:22
@sejas
Copy link
Collaborator Author

sejas commented Mar 29, 2024

Can we delay the last frame in the GIF for an extra 2 seconds where we show the server URL since it's the aha moment and the handoff for our users?

@StevenDufresne, I merged it to quickly. I updated the GIF here: #207

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.

None yet

4 participants