Skip to content

feat: make case sensitive name org field - #262

Merged
johanseto merged 4 commits into
open-release/teak.nelpfrom
jlc/case-sensitive-orgs
Jun 23, 2026
Merged

feat: make case sensitive name org field#262
johanseto merged 4 commits into
open-release/teak.nelpfrom
jlc/case-sensitive-orgs

Conversation

@johanseto

@johanseto johanseto commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Description

This PR helps to create multiple organizations based on the course_org_filter sensitive. Also sometimes the org is not found if only once organizations was created. Then if a user creates courses with an org with different org, the tenant org course not be loaded by the override of get_all_orgs managed by tenant_wise proxy.

eg:

https://github.com/openedx/openedx-platform/blob/release/teak/openedx/core/djangoapps/site_configuration/models.py#L141

because the get_all_org, only return in one case.

def get_all_orgs(cls):

Test

Migrate

Before

2026-06-22_18-44 2026-06-22_18-40

After

2026-06-22_18-46 2026-06-22_18-41

Jira story:

@johanseto
johanseto requested a review from a team as a code owner June 23, 2026 00:01
@johanseto
johanseto requested a review from andrey-canon June 23, 2026 00:01
@johanseto
johanseto force-pushed the jlc/case-sensitive-orgs branch from 0b34c8d to 89b7b48 Compare June 23, 2026 20:28

@andrey-canon andrey-canon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Please open the corresponding PR to include this in the main branch.

When you create it, could you please provide a bit more detail in the description? Right now, I'm personally aware of the context and know what the issue is about, but the current description doesn't completely explain why this change is necessary and under which specific cases the failures occur. It would be great to have that documented for future reference.

@johanseto
johanseto merged commit 3e2c74f into open-release/teak.nelp Jun 23, 2026
7 checks passed
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.

2 participants