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

MyMenu: update theme #7944

Conversation

henriqueiamarino
Copy link
Collaborator

I am testing WordPress Playground to create a PR with the necessary fixes after the Dotorg review.

Also exported as a zip file.

@henriqueiamarino henriqueiamarino changed the title Update theme mymenu MyMenu: update theme Jul 9, 2024
@jasmussen
Copy link
Member

@henriqueiamarino I'm seeing this PR created using a fork similar to how #7928 was created. I guess this is then subject to the same lack of playground link testing as the other PR is?

So perhaps this issue is at play for this one as well: WordPress/wordpress-playground#1575

@bgrgicak can you provide any insights? Thanks for looking.

@jasmussen
Copy link
Member

Asking because this PR and #7917 both appear to be created using the same process, making them much harder to review than, say, #7920.

@henriqueiamarino
Copy link
Collaborator Author

Hey @jasmussen — this is already solved. This is the PR that led me to ask for help. Madhu helped me, and now everything is fine. The theme was merged and reviewed and is live here.

@henriqueiamarino henriqueiamarino deleted the playground-changes-2024-07-09T16-22-53-846Z branch July 15, 2024 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants