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

Multiple Themes: Corrected authors/contributors names #8478

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

iamtakashi
Copy link
Contributor

Fixes #7437

This corrects the authors/contributors' names from the WordPress team to Automattic in multiple themes.

Copy link
Contributor

github-actions bot commented Dec 3, 2024

Preview changes

I've detected changes to the following themes in this PR: Allez, Alter, Blogorama, Boxed Bio, Curriculum, DOS, Grammer One, Inversum, Issue, Leven, Massimo, Messagerie, Modern Business, MyMenu, Organizer, Overlaid, Pieria, Professio, Programme, RAW, Ritratto, Screenplay, Specials, Stage, Texty, The MENU, Varia.
You can preview these changes by following the links below:

I will update this comment with the latest preview links as you push more changes to this PR.

Note

The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.
Child themes are dependent on their parent themes. You will have to install the parent theme as well for the preview to work correctly.

@iamtakashi iamtakashi marked this pull request as draft December 3, 2024 18:06
@iamtakashi iamtakashi marked this pull request as ready for review December 3, 2024 18:54
@iamtakashi
Copy link
Contributor Author

@dsas, I made a PR to fix #7437. I don't think there will be any issue with it, but I wanted you to see it in case there is any concern from the dotcom side.

@dsas
Copy link
Contributor

dsas commented Dec 4, 2024

@dsas, I made a PR to fix #7437. I don't think there will be any issue with it, but I wanted you to see it in case there is any concern from the dotcom side.

Thank you.

No issues that I'm aware of. As far as I remember, the changes won't show up on wordpress.com/themes until they get reindexed into elastic search (i.e. the theme.wordpress.com entry gets updated or someone does a full reindex), I think this is ok.

@iamtakashi iamtakashi merged commit e082cb7 into trunk Dec 4, 2024
2 of 4 checks passed
@iamtakashi iamtakashi deleted the update/author-multiple-themes branch December 4, 2024 12:26
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.

[Bug]: Multiple themes are developed by A8C but are attributed to "The WordPress team"
2 participants