Closed agjohnson closed 1 year ago
Thought I'd share an update on some of the filters here. I'm relying on django-filter for many of the views, and using this UX/UI as a way to consolidate views -- for example, instead of incorporating a list of projects in the team detail UI I will instead link to a filtered list of projects in another UI.
However, there are a few hurdles I've found with django-filter which caused some headaches:
The end solution is ugly in comparison but it is indeed working at least.
The byproduct of this is that I finally found the pattern I wanted in order to surface a search dropdown list for projects/teams/orgs. Currently, these dropdowns use an API call and separate front end code. With the work I did here, we'll be able to do this all from the filter instead.
Next up, we keep talking about the lack of tests for the filters, and I definitely felt this trying to get the filters working correctly. However, I'm not sure how effective I'll be here so I might punt on this and continue with the dashboard changes for now.
Filter changes are here: https://github.com/readthedocs/readthedocs.org/pull/10847
This is what the filters look like currently:
The last of the views that I have been chipping away at are the missing team admin/listing views. I ended up reworking an earlier attempt and making this into a subview of sorts, similar to how the project admin dashboard is a subview of the project dashboard. This mostly makes me want to move all of these views into the organization admin now though, I think that is the correct place for these types of views and there is not much value in duplicating views like organization project listing and team project list when there are filters on the organization project listing.
Parent issue is:
208
This issue will deal specifically with the team and team member pages. I've had a few patterns to work through on these views. This will be a little bit of a collection of a few smaller issues as I work in features from the old templates, including adapting some of the SSO rules that make these pages noop views for the user.
Tasks
organization.teams.all
is correct method to use on team_detail template/team member listing partial/members/
ororganization/member_list.html