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

[GH Request] Repair of the RevSquad Group for frontend-app-ecommerce #1099

Closed
grmartin opened this issue Apr 2, 2024 · 8 comments
Closed
Labels
github-request Request for change to access level or settings in the openedx GitHub organization.

Comments

@grmartin
Copy link

grmartin commented Apr 2, 2024

Firm Name

2U

Urgency

High (< 3 days)

Requested Change

My team is the core Revenue and Purchase Squad at 2U (group openedx/revenue-squad) under @colinbrash. We have write access as far as we know to all of our core repos, however we are missing access to openedx/frontend-app-ecommerce.

Can we have the group permitted to merge as core contributors?

Current Members:

Reasoning

We are the core contributor team to this repo. Initial word during the transition was that we would remain so.

@grmartin grmartin added the github-request Request for change to access level or settings in the openedx GitHub organization. label Apr 2, 2024
@openedx-workflow-automation
Copy link

Thank you for your report! @openedx/axim-oncall will triage within a business day. Simple requests usually take 2-3 business days to resolve; more complex requests could take longer.

@grmartin
Copy link
Author

grmartin commented Apr 2, 2024

This is related to: openedx/frontend-app-ecommerce#379

@e0d
Copy link

e0d commented Apr 2, 2024

@grmartin can you verify that this list of ecommerce maintainers is correct? This repo and other commerce related ones have merge restrictions to support 2U compliance needs. In this case the merge restriction was sublimated by admin access, which has been dialed back. Adding the ecommerce-maintainers team as allowed mergers will fix this, but I wanted to confirm membership first.

FYI, @jristau1984 and @georgebabey

macdiesel Brian Beggs macdiesel
grmartin Glenn R. Martin grmartin
colinbrash Colin Brash colinbrash
iloveagent57 Alex Dusenbery iloveagent57
adamstankiewicz Adam Stankiewicz adamstankiewicz
pshiu Phillip Shiu pshiu
jawad-khan jawad khan jawad-khan
christopappas Chris Pappas christopappas
julianajlk Juliana Kang julianajlk
moeez96 Moeez Zahid moeez96
zubair-ce07 Muhammad Zubair zubair-ce07
mumarkhan999 Muhammad Umar Khan mumarkhan999
JadeyOlivier Jade Olivier JadeyOlivier
MushtaqRossier MushtaqRossier

@grmartin
Copy link
Author

grmartin commented Apr 2, 2024

Getting confirmation.

@grmartin
Copy link
Author

grmartin commented Apr 2, 2024

@e0d: Per @colinbrash this is correct for the moment! No need to cut anyone.

@e0d
Copy link

e0d commented Apr 2, 2024

Can you verify that you can now merge?

@grmartin
Copy link
Author

grmartin commented Apr 2, 2024

@e0d i can and have, Thank you!

@grmartin grmartin closed this as completed Apr 2, 2024
@georgebabey
Copy link

Thank you @e0d! Appreciate the quick response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github-request Request for change to access level or settings in the openedx GitHub organization.
Projects
Status: Done
Development

No branches or pull requests

3 participants