-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Publisher][Superagencies] Add dropdown for child agencies to Set Up Metrics page #1295
Conversation
Hello @mxosman! So, the task says:
As I understand this when we select child agency from this new dropdown we should be delivered to Set Up Metric page of selected child agency, but this contradicts the first requirement since the page we will go to wouldn't have dropdown. Or should we not display dropdown after transition to new page? I'm a little bit confused here, so any clarifications would be very helpful, thank you! |
Hi @nasaownsky! Ah - sorry for the confusion. So, I think the idea is that the dropdown persists the whole time. If we are logged into a superagency and go to the Set Up Metrics page, choosing a child agency should keep you on the same page - just viewing the child agency list of metrics (please correct me if I'm wrong @lilidworkin). Though, this does complicate things as you go past the overview page and try to set metrics for the child agencies... Hmm... just thinking out loud, what if we switch to the child agency after choosing from the dropdown and keep the dropdown visible for the child agency if the user also has access to the superagency? What are your thoughts on that Ilya & @lilidworkin? |
@nasaownsky @mxosman yea this was an oversight on my end -- we should show the dropdown for both superagencies and child agencies! I think that should take care of it. So yea @mxosman basically your second paragraph. |
…to nasaownsky/1275-child-agencies-dropdown
Hi @mxosman! What do you think about my latest changes? Any ideas to iterate this further? Also what about this last paragraph from task description?
|
This is feeling great! One small request after playing around with it - if I'm switched to the child agency, I lose my reference to the superagency, so I have to go back to the search menu. Can the dropdown include the superagency when you switch to a child agency? We can update the copy for the search based on whether or not it's a superagency (if we're in a child agency, we can say "Search for agency" instead) - and even add a "(Superagency)" to the label. Let me know if that feels too weird, Ilya & Lili - I just craved being able to go back to the Superagency after clicking away. |
You can skip this part, @nasaownsky! I'll write up a separate task for it, but it's not blocking for this. It's basically adding groupings to the dropdown, so if a Superagency has a bunch of child agencies, and each child agency has a different sector, the dropdown would group them by sector - so you'd see
We may need help from @forrestkim00 regarding how the groupings should look and solve for something @lilidworkin thought of - child agencies belonging to multiple sectors. Anyway - all this to say that no action needed on this part of the task at this time! |
@mxosman Please, check my changes and let me know your thoughts about it! |
I love that idea @mxosman !
+1 to this, sorry for the confusion! @mxosman lmk if you want me to take a look after you write up the task. Might be a good candidate for Cory actually. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks and works great to me, @nasaownsky! Thank you so much!
Description of the change
Type of change
Related issues
closes #1275
Checklists
Development
This box MUST be checked by the submitter prior to merging:
These boxes should be checked by the submitter prior to merging:
Code review
These boxes should be checked by reviewers prior to merging: