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.

Sign up for GitHub

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

Jump to bottom

Add algorithm for determining primary record in duplicate #105

Open
5 of 13 tasks
Tracked by #103
oubiwann opened this issue Dec 20, 2023 · 0 comments
Open
5 of 13 tasks
Tracked by #103

Add algorithm for determining primary record in duplicate #105

oubiwann opened this issue Dec 20, 2023 · 0 comments
Milestone
0.9.0

Comments

@oubiwann
Copy link
Contributor

oubiwann commented Dec 20, 2023 •
edited
Loading

This should probably be something on the order of:

  • Check that records in group are actually duplicates (according to fields specified for uniquing)
  • Check which one is the most recently updated (this should the primary tie-breaker)
  • Check which one has the highest access count (this should the secondary tie-breaker)
  • If the status differs between two, select active status (tertiary tie-breaker)

Tasks:

  • Revisit password history abstraction
  • Add Column::DuplicateInfo (bool)
  • Include in duplicate listing output
  • Add support for post-processing duplicate records
  • Include history counts when processing records
  • When listing duplicates, in the "Duplicate Info" column
    • set one as primary
    • check to see if the others are identical to existing password history items, and if so, set as in history
    • set all others as duplicate

Part of:
@oubiwann oubiwann mentioned this issue Dec 20, 2023
Open
8 tasks
@oubiwann oubiwann added this to the 0.9.0 milestone Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9.0
Development

No branches or pull requests
1 participant
@oubiwann