search

eduNEXT / edunext-platform

Open edX, the project that powers edunext.co
https://www.edunext.co
GNU Affero General Public License v3.0
29 stars 7 forks source link

["BUG"] FilterUserCourseEnrollmentsByTenant doesn't modify all needed views #737

Closed JuanDavidBuitrago closed 5 months ago

JuanDavidBuitrago commented 1 year ago

Describe the bug Views like api/enrollment/v1/enrollments/ are not filtered correctly.

To Reproduce Steps to reproduce the behavior:

  1. Crea at least two organizations, each one associated to only one tenant, and enroll users to course belonging to those organizations.
  2. Go to 'api/enrollment/v1/enrollments/'
  3. See all enrollment without filters

Expected behavior FIlter enrollments using the current site.

Additional context This is the view I found, but there might be others

JuanDavidBuitrago commented 1 year ago

@Ian2012 here is your reported issue.

Ian2012 commented 1 year ago

@JuanDavidBuitrago is this a better place for this issue? 🤔

I forgot to mention that this issue is related to the TenantAwareCourseEnrollmentFilter defined on eox-core

JuanDavidBuitrago commented 1 year ago

FilterUserCourseEnrollmentsByTenant is associated with the location of the filter, but I will check it.

santiagosuarezedunext commented 1 year ago

I'm going to put this issue in prioritized and we'll do it when we attack FFI-8

MaferMazu commented 5 months ago

I talked with @Ian2012 about this card, and he is not having this problem anymore because he has not been using the feature recently. I will close this because he doesn't need to pursue this, and the description talked about api/enrollment/v1/enrollments/ an API from the edx-platform but involved eox-core and tenant, which makes this issue unclear.

If someone has this problem or needs to pursue this, please feel free to reopen this issue or open a new one in the required repository.