Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Editors linked to channels on Admin page is yielding a different result #3905

Closed
laurenlichtman opened this issue Jan 11, 2023 · 6 comments · Fixed by #3958
Closed

Editors linked to channels on Admin page is yielding a different result #3905

laurenlichtman opened this issue Jan 11, 2023 · 6 comments · Fixed by #3958
Assignees
Labels
P1 - important Priority: High impact on UX

Comments

@laurenlichtman
Copy link

laurenlichtman commented Jan 11, 2023

Observed behavior

On the admin panel on the "Channels tab", when I click on "can edit" for a particular channel, the new page that pops out has a number of users that does not match the number listed on the "Channels tab" next to "can edit." I tested this for multiple channels.

Steps to reproduce

  1. Sign in as an admin
  2. Open the Channels view/tab in the admin
  3. Filter the channels using a keyword
  4. Click the icon button in the 'Editors' column to view a channel's editors
  5. Observe it shows more users than totaled in the 'Editors' column
Screenshare.-.2023-01-10.3_59_34.PM.mp4
@bjester bjester added this to the Studio: next major release milestone Jan 24, 2023
@bjester bjester added the P1 - important Priority: High impact on UX label Jan 24, 2023
@akash5100
Copy link
Contributor

Hi, I would like to work on this issue and contribute to Learning Equality - Studio project. Can you please provide more information on the steps I should take to fix the discrepancy in the number of users listed next to "can edit" on the Channels tab and the new page that pops out when clicking on "can edit" for a particular channel?

Thank you

@vkWeb
Copy link
Member

vkWeb commented Feb 7, 2023

@bjester I have not been able to reproduce this locally on unstable. @bjester can you try on your end?

@akash5100 Hi and welcome! Meanwhile look at https://github.com/learningequality/studio/milestone/13 for more issues that you might be interested in.

@vkWeb
Copy link
Member

vkWeb commented Feb 7, 2023

@bjester Also, I guess #3611 aimed to fix the same bug mentioned on this issue?

@bjester
Copy link
Member

bjester commented Feb 7, 2023

I've updated the issue description with clear steps to reproduce. It has been confirmed that #3611 didn't fix this entirely.

@akash5100
Copy link
Contributor

I have tried to reproduce the issue by the steps you provided.

However, I have noticed that the issue does not occur in the development environment, which only has 4 users (A, B, C & Admin). I am unable to reproduce the issue with the limited number of users in the development environment.

I would like to suggest adding a bunch of dummy users and testing the issue again to see if this can help us better understand and resolve the issue.

Could you please let me know if my suggestion of adding dummy users in bulk is viable and if there are any alternative methods to better understand and resolve the issue?

Thank you :)

@bjester
Copy link
Member

bjester commented Feb 8, 2023

@akash5100 Here are steps to reproduce the issue in local development:

  1. Sign in as [email protected] with password b
  2. Create a new channel with name published
  3. Sign out
  4. Sign in as the admin user [email protected] with password a
  5. Open the Channels page in the Administration section
  6. Find the pre-existing channel called Published Channel which is public (has green dot)
  7. Click on the button in 'Editors' column for the Published Channel
  8. Observe User B is listed as an editor but isn't actually an editor of Published Channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 - important Priority: High impact on UX
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants