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 CONTRIBUTING.md #5097

Closed
wants to merge 1 commit into from
Closed

Conversation

Christinadobrzyn
Copy link
Contributor

@Christinadobrzyn Christinadobrzyn commented Sep 6, 2021

Revised some wording under the Finding Jobs section - specific to when a contributor qualifies for a bonus and how to submit a proposal.

Details

  • Revised some wording under the Note section here
    image

  • Revised some wording in this section
    image

Fixed Issues

NA - no issues fixed, just some rewording in the above sections.

Tests

NA

--->

QA Steps

NA
--->

Tested On

NA

Screenshots

See Above

Web

NA

Mobile Web

NA

Desktop

NA

iOS

NA

Android

NA

Revised some wording under the Finding Jobs section - specific to when a contributor qualifies for a bonus and how to submit a proposal.
@Christinadobrzyn Christinadobrzyn requested a review from a team as a code owner September 6, 2021 07:40
@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2021

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request

@MelvinBot MelvinBot requested review from mountiny and removed request for a team September 6, 2021 07:40
@mountiny
Copy link
Contributor

mountiny commented Sep 6, 2021

@Christinadobrzyn Hi! You will need to sign the CLA assistant so we can merge this.

Just copy this text:

I have read the CLA Document and I hereby sign the CLA

into a new comment here and post it. That should do it 😊

@mallenexpensify
Copy link
Contributor

@Christinadobrzyn can you review the changes being made here to see if they're covered?
#4996
If not, can you update that issue with the suggested edits?

@Christinadobrzyn Christinadobrzyn self-assigned this Sep 7, 2021
@Christinadobrzyn
Copy link
Contributor Author

Christinadobrzyn commented Sep 7, 2021

Ah yikes! Thanks for catching that Matt!

Yeah I don't want to disrupt the updates that Matt proposed. Because his pull request has many more updates and mine are very minor, I'm going to close this and propose the updates in his pull request.

@Christinadobrzyn Christinadobrzyn removed their assignment Sep 7, 2021
@Christinadobrzyn
Copy link
Contributor Author

Proposed these changes in Matts PR.

@roryabraham roryabraham deleted the Christinadobrzyn-patch-1 branch June 8, 2023 01:22
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