Skip to content
This repository has been archived by the owner on Nov 18, 2024. It is now read-only.

Remove contributor section from PR template, add pattern creation guidelines to readme. #45

Merged
merged 5 commits into from
Aug 17, 2024

Conversation

juanfra
Copy link
Member

@juanfra juanfra commented Aug 16, 2024

Description

  • Removing contributors section from PR template as it's now handled via the props bot action.
  • Adding Pattern creation guidelines section to the readme

@juanfra juanfra requested a review from carolinan August 16, 2024 10:28
Copy link

github-actions bot commented Aug 16, 2024

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: juanfra <[email protected]>
Co-authored-by: aaronjorbin <[email protected]>
Co-authored-by: huzaifaalmesbah <[email protected]>
Co-authored-by: shail-mehta <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

README.md Outdated Show resolved Hide resolved
@shail-mehta
Copy link
Member

There is small typo in README.md Line no 69

texdomain it should be textdomain

Copy link
Contributor

@huzaifaalmesbah huzaifaalmesbah left a comment

Choose a reason for hiding this comment

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

Since this theme is a TT5 theme, the textdomain here should be TT5.

README.md Outdated

```
<!-- wp:image {"id":125,"sizeSlug":"large","linkDestination":"none"} -->
<figure class="wp-block-image size-large"><img src="http://wp-stable.test/wp-content/themes/twentytwentyfour/assets/images/project.webp" alt="" class="wp-image-125"/></figure>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think 'twentytwentyfour' will be 'twentytwentyfive'

README.md Outdated

```
<!-- wp:image {"sizeSlug":"large","linkDestination":"none"} -->
<figure class="wp-block-image size-large"><img src="<?php echo esc_url( get_template_directory_uri() ); ?>/assets/images/project.webp" alt="<?php echo esc_attr_x( 'Picture of a building', 'Alt text for project picture', 'twentytwentyfour' ); ?>"/></figure>
Copy link
Contributor

Choose a reason for hiding this comment

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

'twentytwentyfour' to 'twentytwentyfive'

README.md Outdated

and

`<!-- wp:template-part {"slug":"header-portfolio","theme":"twentytwentyfour","area":"header","tagName":"header"} /-->`
Copy link
Contributor

Choose a reason for hiding this comment

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

'twentytwentyfour' to 'twentytwentyfive'

@juanfra
Copy link
Member Author

juanfra commented Aug 17, 2024

Thanks for the reviews. Merging this one before we start getting contributions for patterns.

@juanfra juanfra merged commit 19a1aa1 into trunk Aug 17, 2024
1 check passed
@juanfra juanfra deleted the fix/contributors branch August 17, 2024 11:22
@juanfra juanfra mentioned this pull request Aug 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants