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

Pull Org Data from the Export CSV instead of orgs.yaml #199

Open
feanil opened this issue Sep 26, 2022 · 2 comments
Open

Pull Org Data from the Export CSV instead of orgs.yaml #199

feanil opened this issue Sep 26, 2022 · 2 comments
Assignees

Comments

@feanil
Copy link
Contributor

feanil commented Sep 26, 2022

Context: openedx/openedx-webhooks-data-schema#15

We have stopped updating orgs.yaml because it is now redundant with the data we're getting from the salesforce export file. We need to update the openedx-webhooks code to read orgs out of the salesforce export CSV instead of orgs.yaml or potentially in addition to orgs.yaml.

@feanil
Copy link
Contributor Author

feanil commented Sep 26, 2022

Looking at the code in openedx-webhooks, it looks like we use orgs.yaml to determine if an organization is "internal" or not. The behavior of the bot is different for "internal" groups vs external groups.

@feanil
Copy link
Contributor Author

feanil commented Sep 26, 2022

Ideal state would be to have an "Internal" Column in the salesforce export.

We also can't do this in a sane way until we've cleaned up how we use people.yaml first. A bunch of the validation for that file assumes an orgs.yaml exists and is coherent with the people.yaml

I'm gonna write up a task to understand what we still need people.yaml for and what it would take to get rid of it.

Blocked by #200

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

No branches or pull requests

1 participant