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

button "Review ignored people" to review again ignored/hidden persons #747

Merged
merged 1 commit into from
May 15, 2024

Conversation

wronny
Copy link
Contributor

@wronny wronny commented May 1, 2024

Hi,
I agree on #735 - currently ignored/hidden people can only be shown again by making a change within the database. Therefore I added a button "Review ignored people" to review again ignored/hidden persons. This button only appears when ignored/hidden people are found.
Screenshot 2024-05-01 221643
Screenshot 2024-05-01 221732
I hope this will be helpful.

…eople. This button only appears when ignored/hidden people are found.

Signed-off-by: wronny <[email protected]>
@matiasdelellis
Copy link
Owner

Ohh. Great addition @wronny

These days I will do a review. At first glance I like it, and I just don't understand that sleep(1). 🤔 but if everything is fine I'll probably merge it and then make some changes. 😉

@wronny
Copy link
Contributor Author

wronny commented May 2, 2024

I just don't understand that sleep(1)

if both - new and ignored persons are found - I experienced a random order of both buttons. To delay checking for ignored/hidden persons was a quick workaround to have "Review ignored people" always as second button.
Of course feel free to do any changes. This was just my proposal.

@matiasdelellis
Copy link
Owner

Hi @wronny
I'm going to merge it as is and make some changes..

Thanks again..

@matiasdelellis matiasdelellis merged commit abc830a into matiasdelellis:master May 15, 2024
5 of 32 checks passed
@wronny wronny deleted the show-ignored-clusters branch May 15, 2024 20:14
matiasdelellis added a commit that referenced this pull request May 17, 2024
It is a minimal improvement on how the application should always have
been. Instead of getting the information at the start of the
application, we do it only when we know there are values and the user
wants to do something with it.

This also applies to the user's enablement value as well as to the set
of new people to recognize.
@matiasdelellis
Copy link
Owner

Done.. See last commit..
Some things still need to be improved (Modernize the entire script? 🥺), but this was the minimum I wanted to improve now.

Thanks again. With this tomorrow I will upload a new release. 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants