Skip to content

Update/community files#37

Merged
dkotter merged 6 commits intodevelopfrom
update/community-files
May 14, 2025
Merged

Update/community files#37
dkotter merged 6 commits intodevelopfrom
update/community-files

Conversation

@jeffpaul
Copy link
Copy Markdown
Member

@jeffpaul jeffpaul commented May 2, 2025

Description of the Change

This PR fixes some issues with displaying Jobber instead of 10up, fixes license references, adds repo badges.

Before merging this in, I'd like to get input from Jobber on what email to properly list here:

<opensource@10up.com>.

As those CoC complaints should not come to 10up, but rather Jobber. THIS HAS BEEN UPDATED

How to test the Change

Most changes can be previewed with a markdown or github previewer.

Changelog Entry

Added - Plugin documentation.

Credits

Props @jeffpaul.

Checklist:

@jeffpaul jeffpaul added this to the 1.0.0 milestone May 2, 2025
@jeffpaul jeffpaul self-assigned this May 2, 2025
@github-project-automation github-project-automation bot moved this to Backlog in 10up Jobber May 2, 2025
@jeffpaul jeffpaul requested a review from dkotter May 2, 2025 19:40
@github-actions github-actions bot added the needs:code-review This requires code review. label May 2, 2025
dkotter
dkotter previously approved these changes May 2, 2025
@jeffpaul jeffpaul moved this from Backlog to Code Review (Jobber) in 10up Jobber May 5, 2025
@jeffpaul jeffpaul moved this from Code Review (Jobber) to UAT Approved in 10up Jobber May 5, 2025
@dkotter
Copy link
Copy Markdown
Collaborator

dkotter commented May 12, 2025

Looks like the automated review from the WordPress.org team has flagged issues around ownership. I think this PR addresses everything they're concerned with so would be ideal to get this merged in.

@jeffpaul Are we still waiting on this:

Before merging this in, I'd like to get input from Jobber on what email to properly list here

@jeffpaul
Copy link
Copy Markdown
Member Author

I'd like to get Jobber to provide an email for the CoC; @vikrampm1 mind leaning on Andrew to get that answered so we can add here and get this merged in? (If that drags more than a couple days let's merge here and track that separately as a follow-up item).

@vikrampm1
Copy link
Copy Markdown
Contributor

Here is the email to update: integrations@getjobber.com

@dkotter dkotter merged commit c97a94d into develop May 14, 2025
8 of 9 checks passed
@dkotter dkotter deleted the update/community-files branch May 14, 2025 16:14
@github-project-automation github-project-automation bot moved this from UAT Approved to Merged in 10up Jobber May 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants