search

eduNEXT / eox-tenant

Plugin for managing multiple tenants (organizations) within a single Open edX instance.
GNU Affero General Public License v3.0
7 stars 4 forks source link

perf: added palm support and changed default settings to always use e… #190

Closed Asespinel closed 1 year ago

Asespinel commented 1 year ago

This PR mainly updates the requirements for Palm support, and it modifies the setting USE_EOX_TENANT = True as I beleive we should have this as a default if eox_tenant is being installed and ready to use.

Testing instructions

  1. Install this version of eox-tenant in a palm environment.
  2. Create routes and tenant configs accordingly as a super user
  3. For example: Screenshot from 2023-11-16 20-36-29
  4. If you go to the specified route you should read the "site1" platform name.
  5. Other tests cases can be found at: https://docs.google.com/document/d/1921hOajxAYrdin_thEZ005aQBwPNoj2zsbgkf3q1Mjc/edit
Asespinel commented 1 year ago

Hi @MaferMazu Thanks for the comments they are always greatly appreciated.

Improvements I left an Issue regarding the JWT that we can tackle in our next sprints.

Requirements I revisited the constraints file and changed it accordingly. Please let em know if you have additional comments so I can modify the file accordingly

Documentation I changed the readme witht he right version of eox tenant and changed the init file as well. Regarding the drive documentation, Im going to take another look to the changes and I'll work on it.

I also modified the PR title to better suit the objective of the PR