openedx / edx-documentation

http://docs.edx.org
Other
161 stars 333 forks source link

docs: Added limited staff role #2185

Closed bpedx closed 9 months ago

bpedx commented 9 months ago

Add a description of your changes with links to any relevant material.

Date Needed (optional)

If the release date of a feature is known or estimated, provide it to give reviewers guidance on turnaround time.

Reviewers

Possible roles follow. The PR submitter checks the boxes after each reviewer finishes and gives :+1:.

FYI: Tag anyone else who might be interested in this PR here.

Testing

HTML Version (optional)

Sandbox (optional)

Post-review

openedx-webhooks commented 9 months ago

Thanks for the pull request, @bpedx! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

:warning: We can't start reviewing your pull request until you've submitted a signed contributor agreement or indicated your institutional affiliation. Please see the CONTRIBUTING file for more information. If you've signed an agreement in the past, you may need to re-sign. See The New Home of the Open edX Codebase for details.

Once you've signed the CLA, please allow 1 business day for it to be processed. After this time, you can re-run the CLA check by adding a comment here that you have signed it. If the problem persists, you can tag the @openedx/cla-problems team in a comment on your PR for further assistance.

mphilbrick211 commented 9 months ago

Hi @bpedx! Thanks for this contribution. Are you a 2U/edX employee? If so, you'll need to be added to their entity CLA (have your manager reach out to oscm@axim.org. If you're contributing as an individual, you'll need to fill out the individual CLA as indicated above.

bpedx commented 9 months ago

Hi Michelle,

It’s Ben Piscopo on the 2U Product team. Ned B and Jeremy R both submitted info for me today.

Do you mind also checking for me under @.*** ?

Thanks, Ben

On Thu, Sep 14, 2023 at 16:33 Michelle Philbrick @.***> wrote:

Hi @bpedx https://github.com/bpedx! Thanks for this contribution. Are you a 2U/edX employee? If so, you'll need to be added to their entity CLA (have your manager reach out to @.*** If you're contributing as an individual, you'll need to fill out the individual CLA as indicated above.

— Reply to this email directly, view it on GitHub https://github.com/openedx/edx-documentation/pull/2185#issuecomment-1720108718, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGEA6F362ITKNWWWW422ZM3X2NS3NANCNFSM6AAAAAA4YTUVJA . You are receiving this because you were mentioned.Message ID: @.***>

feanil commented 9 months ago

FYI, I just updated Ben's records so the CLA bot should update overnight tonight.

mphilbrick211 commented 9 months ago

FYI, I just updated Ben's records so the CLA bot should update overnight tonight.

Thanks, @feanil! FYI @bpedx

feanil commented 9 months ago

No, you are covered by the 2U Entity Agreement.

bpedx commented 9 months ago

Nevermind, the checklist just updated itself... but now "this branch is out-of-date with the base branch". Should I Update Branch or should I Update with Rebase? @feanil

mphilbrick211 commented 9 months ago

Hi @bpedx! Rebasing should fix it.