Skip to content

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.

@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.

@cellio cellio requested a review from ArtOfCode- December 26, 2024 20:21
@ArtOfCode- ArtOfCode- merged commit 2b33075 into develop Jan 3, 2025
7 checks passed
@ArtOfCode- ArtOfCode- deleted the 0valt/1489/system-user branch January 3, 2025 02:32
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

4 participants