Rework community views to operate on Person instead of User #5859
Labels
accepted
component: community/
enhancement
New feature or request
feat/user2person
Refactoring objects that point into User that should have pointed into Peron
medium
Comments
jennifer-richards
added
enhancement
rjsparks
added
the
feat/user2person
pselkirk
added a commit
to pselkirk/datatracker
that referenced
this issue
Jun 29, 2023
rjsparks
added a commit
that referenced
this issue
Jan 24, 2024
* refactor: change references from User to Person (#5821) * refactor: Change CommunityList reference from User to Person * refactor: Convert more user references to person * refactor: Change augment_docs_and_user_with_user_info to person * refactor: Change Nomination and Feedback references from User to Person * refactor: Change a few test case function signatures to be more pythonic * refactor: Harmonize how profile and photo views look up email_or_name * refactor: Rework community views to operate on Person instead of User (#5859) * test: Update tests to try all of the person's emails and aliases * fix: Recode a test case to avoid an exception if there's Unicode in the URL This only happens using the form-filling and submission feature of WebTest, which is only used in this one test case, so just it rip out. * test: Add duplicate-person tests * fix: If there are multiple matching users, prefer the logged-in one. * chore: We no longer use WebTest, so don't include it. * fix: Address review comments * fix: case-insensitive person name or email matching (#6096) * chore: Renumber migrations * fix: Update merged code so tests pass (#6887) * fix: Use refactored method * fix: Don't assume user has person * fix: Use new view param name * chore: Drop community lists w/o person; cleanup (#6896) * fix: Don't assume user has person * fix: user->person in update_community_list_index.py * feat: Remove CommunityLists without Person * refactor: Speed up nomcom migrations --------- Co-authored-by: Paul Selkirk <[email protected]> Co-authored-by: Jennifer Richards <[email protected]>
|
Fixed by #6024 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
The views in
ietf.community.viewstake ausernameparameter and use these to look up the relevantCommunityList, etc. This should be refactored to be aPerson-centric interface, looking up the relevantPersonbased on their email address (à la the/person/view) and then making use of the refactoredCommunityListmodel that now links to aPerson.This applies to the
feat/user2personbranch at the moment.Code of Conduct
The text was updated successfully, but these errors were encountered: