If Tiers app not enabled return all orgs as active - Githubissues

appsembler / edx-platform

Appsembler Tahoe fork of Open edX. Branches: `main` for Juniper and `hawthorn/main` for Hawthorn.

https://appsembler.com/products/tahoe/

GNU Affero General Public License v3.0

13 stars 15 forks source link

If Tiers app not enabled return all orgs as active #1334

Closed bryanlandia closed 1 year ago

bryanlandia commented 1 year ago

Change description

Mostly for devstacks. If Tiers app is not enabled, don't try to check its database for active Organizations, just return all. Fixes devstack Studio errors and simplifies config.

Type of change

[x] Bug fix (fixes an issue)
[ ] New feature (adds functionality)

Related issues

Checklists

Development

[x] Lint rules pass locally
[x] Application changes have been tested thoroughly
[x] Automated tests covering modified code pass

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Code review

[x] Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
[x] "Ready for review" label attached and reviewers assigned
[ ] Changes have been reviewed by at least one other contributor
[ ] Pull request linked to task tracker where applicable

github-actions[bot] commented 1 year ago

Checking git merge conflicts against https://github.com/edx/edx-platform.git

Comparing with	`open-release/nutmeg.master`
Benchmark conflicts with `main`	292
Current conflicts	292
Summary	Good work! No added conflicts.

Comparing with	`master`
Benchmark conflicts with `main`	300
Current conflicts	300
Summary	Good work! No added conflicts.