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

Publish CC Data In salesforce-export.csv #207

Closed
5 of 9 tasks
feanil opened this issue Oct 12, 2022 · 4 comments
Closed
5 of 9 tasks

Publish CC Data In salesforce-export.csv #207

feanil opened this issue Oct 12, 2022 · 4 comments

Comments

@feanil
Copy link
Contributor

feanil commented Oct 12, 2022

See #200 and #227 for some background.

Things we probably need to do as a part of this work:

  • Add a new fields to salesforce to track CC related data.
    • CCs are tracked at the repo level and at the org level so we may need both but might be able to get away with just the repo field to start with.
  • Add/Update documentation around addition and removal of CCs and what to do in salesforce at those times.
  • Update the export so that the new fields are being publish to the CSV.
  • Update openedx-webhooks to pull this new data from the CSV.
  • Update openedx-webhooks to pull org data from the CSV and drop reading data from orgs.yaml
  • Add back core-contributor-specific OSPR bot comments. This feature was removed in Remove history, core contributor status, and people.yaml #251, since the OSPR bot was relying on an outdated data source (people.yaml) to leave such comments.
  • Remove the people.yaml and orgs.yaml files from the openedx-webhooks-data repo.
  • Figure out if we still need repo-tools-data-schema and update it as necessary to work with the new data setup.
    • With validation happening in salesforce we may not need repo-tools-data-schema.
@e0d
Copy link

e0d commented Jun 9, 2023

@feanil any value left here?

@feanil
Copy link
Contributor Author

feanil commented Jun 12, 2023

We still need this to fully cleanup our reliance on the old people.yaml file. We could also just remove this feature from the bot but one of those two things needs to happen still.

@kdmccormick
Copy link
Member

@nedbat removed our reliance on people.yaml in #251, and by necessity removed all the core-contributor-aware features (notably: special PR comments for CCs) that depended on people.yaml.

If we implement this ticket, we'll need to put those CC PR comments back. I updated the task list in the ticket's description.

@kdmccormick
Copy link
Member

I broke the remaining parts of this out into: #312

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

3 participants