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

Unmanaged organization should not appear in any 'select organization' list #260

Closed
1 task done
pengyin-shan opened this issue Nov 16, 2022 · 16 comments
Closed
1 task done
Assignees

Comments

@pengyin-shan
Copy link

pengyin-shan commented Nov 16, 2022

Will remove unmanaged organizations from the organization type-in list when the user creates a new account:

Screenshot 2022-11-16 at 4 31 26 PM

Meanwhile, @isawil can you pls take a look at the whole application (you can do this on uat: https://uat.library.ualberta.ca:3000/) and see if anywhere else also has an organizational dropdown list? I'll modify them to show managed org only.

As a note, currently, the 'Create Plan' page and 'Statistics' page have the organizational list only showing managed org.

@pengyin-shan pengyin-shan self-assigned this Nov 16, 2022
@shilowil
Copy link
Collaborator

Sorry this is so late! I'm also seeing a drop-down list on the "Templates" page, but am unsure if this is just something visible to the superadmin.

@pengyin-shan
Copy link
Author

Sorry this is so late! I'm also seeing a drop-down list on the "Templates" page, but am unsure if this is just something visible to the superadmin.

I see, can you put a screenshot here? If that's also a place that needs unmanaged organization to be removed from the list, I'll apply changes to it.

@shilowil
Copy link
Collaborator

Yup -- here it is:

image

@pengyin-shan
Copy link
Author

Yup -- here it is:

image

Sorry where is this screenshot coming from? You said in Templates page right? How you got there?

@pengyin-shan
Copy link
Author

Yup -- here it is:
image

Sorry where is this screenshot coming from? You said in Templates page right? How you got there?

Oh never mind, I found it. The templates is OK since it doesn't show unmanaged organization. For example, you won't see OMAFRA many times in this list

@pengyin-shan
Copy link
Author

Yup -- here it is:
image

Sorry where is this screenshot coming from? You said in Templates page right? How you got there?

Oh never mind, I found it. The templates is OK since it doesn't show unmanaged organization. For example, you won't see OMAFRA many times in this list

Hold on..I noticed my local version is different from the remote version. I'll check if this is due to some uncommitted changes

@pengyin-shan
Copy link
Author

As a side note for navigation, the form on the Contributor page does not have unmanaged org listed:
Screenshot 2022-11-28 at 5 36 28 PM.

Will keep this in mind.

@pengyin-shan
Copy link
Author

@isawil : when you have time, pls test the following in uat:

  1. Contributors should be able to add/switch organizations now.

  2. All org selectors (in registration, create plan,... all pages except the creating new organization page) should not include any unmanaged organization.

  3. Adding a new organization page will have a SLOWER org selector, because it is configured to query external organizations as well. Only this page should include unmanaged organizations because we want the super admin to be reminded that such an organization already exists.

@pengyin-shan
Copy link
Author

@isawil : when you have time, pls test the following in uat:

  1. Contributors should be able to add/switch organizations now.
  2. All org selectors (in registration, create plan,... all pages except the creating new organization page) should not include any unmanaged organization.
  3. Adding a new organization page will have a SLOWER org selector, because it is configured to query external organizations as well. Only this page should include unmanaged organizations because we want the super admin to be reminded that such an organization already exists.

Pls let me know if any org selector doesn't follow this rule.

@shilowil
Copy link
Collaborator

shilowil commented Dec 9, 2022

Results of the test:

  1. This works -- thanks!
  2. I'm not seeing unmanaged orgs on any of these pages, although the orgs that show up differ based on page (I couldn't add UofA when selecting a contributor organization, though I could in all other pages)
  3. Org selector is slower

@pengyin-shan
Copy link
Author

Thanks @isawil ! Yes adding a new organization page did have a slower selector since it needs to query ror.org, so it is expected. You will need to enter the full name, wait a little bit, then see if the result is returned or not. The regular users won't be affected by this behavior, since we only want the super admin to create org.

The contributor page also works, which is fine.

Where did you encounter the U of A issue? Is this page?
Screenshot 2022-12-09 at 3 06 30 PM

@pengyin-shan
Copy link
Author

@isawil as I noted today at our service team meeting, I removed this version from uat because I need to test 3.1.0 migration. So if you want to test this again, let me know and I'll re-deploy this back to uat.

@shilowil
Copy link
Collaborator

shilowil commented Jan 6, 2023

Hi Wendy -- sorry for this delay! Following up on this now. If it doesn't interfere with the 3.1.0 migration, could you re-deploy to UAT?

@pengyin-shan
Copy link
Author

Hi Wendy -- sorry for this delay! Following up on this now. If it doesn't interfere with the 3.1.0 migration, could you re-deploy to UAT?

Sorry this version has been replaced by my 3.1.0 test..don't worry, once 3.1.0 is deployed I'll merge this change, and update you once it is ready for testing

@shilowil
Copy link
Collaborator

shilowil commented Jan 6, 2023

No worries! Thanks.

@pengyin-shan
Copy link
Author

Close this issue. Applied changes to version 3.1.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants