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

[candidate_list] Remove meaningless Latest_Visit_Status column #7759

Merged
merged 2 commits into from Oct 14, 2022

Conversation

driusan
Copy link
Collaborator

@driusan driusan commented Oct 29, 2021

The max alphabetical visit doesn't mean anything. (See resolved issues for explanation.)

Resolves #5115, #7093

@driusan driusan added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Oct 29, 2021
@driusan
Copy link
Collaborator Author

driusan commented Oct 29, 2021

Not sure if this should target v24 or be kept in draft mode until after the release since we've already tested it with the meaningless behaviour.

@driusan
Copy link
Collaborator Author

driusan commented May 17, 2022

@xlecours since you filed #5115 can you review/test this?

@maltheism maltheism added the Passed Manual Tests PR has undergone proper testing by at least one peer label Oct 14, 2022
Copy link
Member

@maltheism maltheism left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@driusan passed manual testing

@driusan driusan merged commit ff4cb6c into aces:main Oct 14, 2022
@ridz1208 ridz1208 added this to the 25.0.0 milestone Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Passed Manual Tests PR has undergone proper testing by at least one peer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Candidate_list] Latest Visit Status value is ambiguous
3 participants