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

fix: return by error removed verified users #644

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

shpingalet007
Copy link
Collaborator

@shpingalet007 shpingalet007 commented Aug 24, 2022

Standards checklist:

  • The PR title is descriptive.
  • The PR doesn't replicate another PR which is already open.
  • The PR has self-explained commits history.
  • The code is mine or it's from somewhere with an Apache-2.0 compatible license.
  • The code is efficient, to the best of my ability, and does not waste computer resources.
  • The code is stable and I have tested it myself, to the best of my abilities.
  • If the code introduces new classes, methods, I provide a valid use case for all of them.

Changes:

  • Returned lost verified users entries

Other comments:

Some verified users were suddenly removed from verified list. It was occurred due to some list optimization (there were duplicated entries). In this pull request, I return them.

Full list of lost verifications returned

  • SekretyPodnebesnoy
  • Rostislav_z
  • slavradio
  • timcast_irl
  • tim_pool
  • RedPill78

@shpingalet007 shpingalet007 added the contribution type: 🐛 fix Something isn't working label Aug 24, 2022
@shpingalet007 shpingalet007 self-assigned this Aug 24, 2022
@shpingalet007 shpingalet007 changed the title Return by error removed verified users fix: return by error removed verified users Aug 24, 2022
@andyoknen andyoknen self-requested a review August 24, 2022 19:50
@shpingalet007 shpingalet007 merged commit b03e52c into master Aug 24, 2022
@shpingalet007 shpingalet007 deleted the fix-verified-users branch June 13, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution type: 🐛 fix Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants