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

Add seeding of the system user's community users #1490

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

Oaphi
Copy link
Member

@Oaphi Oaphi commented Dec 16, 2024

closes #1489

blocks #1487

@Oaphi Oaphi self-assigned this Dec 16, 2024
@Oaphi Oaphi requested review from a team and cellio December 16, 2024 10:05
@Oaphi
Copy link
Member Author

Oaphi commented Dec 16, 2024

Note: I opted for trust level equal to 4, but not sure what's the trust level for the user's community users on production communities - happy to adjust if the assumption is incorrect.

@cellio
Copy link
Member

cellio commented Dec 16, 2024

Note: I opted for trust level equal to 4, but not sure what's the trust level for the user's community users on production communities - happy to adjust if the assumption is incorrect.

In production "trust level" is NULL, but mine and Art's are both 5. I'm not sure what we use this for today other than category read access (which System wouldn't care about). I'd guess we should go with NULL, and I'll defer if another reviewer says something different.

db/seeds.rb Show resolved Hide resolved
@Oaphi
Copy link
Member Author

Oaphi commented Dec 16, 2024

Note: I opted for trust level equal to 4, but not sure what's the trust level for the user's community users on production communities - happy to adjust if the assumption is incorrect.

In production "trust level" is NULL, but mine and Art's are both 5. I'm not sure what we use this for today other than category read access (which System wouldn't care about). I'd guess we should go with NULL, and I'll defer if another reviewer says something different.

Perfect, thanks! I'll change it to nil then - level 4 was just a placeholder value until someone with production access could verify.

@cellio
Copy link
Member

cellio commented Dec 18, 2024

LGTM. Needs review by a better Rubyist.

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.

System user's community users are not seeded
2 participants