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

Update README #3332

Merged
merged 3 commits into from
Sep 15, 2019
Merged

Conversation

kennyadsl
Copy link
Member

Description
Update the README with the latest things happening on Open Collective.

Checklist:

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have updated Guides and README accordingly to this change (if needed)
  • I have added tests to cover this change (if needed)

@kennyadsl kennyadsl self-assigned this Sep 12, 2019
@kennyadsl kennyadsl changed the title Kennyadsl/update footer Update README Sep 12, 2019
README.md Outdated
### Key Stakeholders
Support this project by becoming a Key Stakeholder. Your logo will show up here with a link to your website. [Become a Key Stakeholder](https://opencollective.com/solidus#key-stakeholder).
### Ambassadors
Support this project by becoming a Solidus Ambassadors. Your logo will show up here with a link to your website. [Become an Ambassador](https://opencollective.com/solidus).
Copy link
Contributor

Choose a reason for hiding this comment

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

I would make this singular instead of plural. Solidus Ambassadors -> Solidus Ambassador

Suggested change
Support this project by becoming a Solidus Ambassadors. Your logo will show up here with a link to your website. [Become an Ambassador](https://opencollective.com/solidus).
Support this project by becoming a Solidus Ambassador. Your logo will show up here with a link to your website. [Become an Ambassador](https://opencollective.com/solidus).

Copy link
Member Author

@kennyadsl kennyadsl Sep 13, 2019

Choose a reason for hiding this comment

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

Fixed, thank you!

This reflects the changes that we'd done in Open Collective
groups naming.
- rename Open Collective tiers following the new naming structure
- add a missing Open Collective tier
- split badges in 2 blocks, one for tech stuff and one for community
Copy link
Contributor

@ericsaupe ericsaupe left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

Copy link
Contributor

@jacobherrington jacobherrington left a comment

Choose a reason for hiding this comment

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

Thanks @kennyadsl!

@jacobherrington jacobherrington merged commit 29acc0d into solidusio:master Sep 15, 2019
@kennyadsl kennyadsl deleted the kennyadsl/update-footer branch September 16, 2019 08:17
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.

3 participants