Add reverse search (#40) #54

Open
brickedsalamander wants to merge 2 commits from brickedsalamander/iseeyour.cash:reverse-search into main
Collaborator

This PR adds a reverse search to the existing searchbar. A visitor can search for an address and the matching username and other addresses are shown, see #40.
The behavour in detail:

  • Every input is checked if it matches the pattern of one of the supported addresss or a Monero address.
  • If it is a Monero address, a "Nice Try" message is shown.
  • If the address is known to the database, i.e. associated with a username, the info about the user is shown with a message stating that the entered address is owned by the user.
  • If the address is not known in the database, only its balace is shown.
  • If the address matches multiple patterns, the balance on all blockchains is shown. (not tested yet, currently supported currencies don't have matching patterns)

Tested on a local instance. Someone else should test it before merging.

This PR adds a reverse search to the existing searchbar. A visitor can search for an address and the matching username and other addresses are shown, see #40. The behavour in detail: - Every input is checked if it matches the pattern of one of the supported addresss or a Monero address. - If it is a Monero address, a "Nice Try" message is shown. - If the address is known to the database, i.e. associated with a username, the info about the user is shown with a message stating that the entered address is owned by the user. - If the address is not known in the database, only its balace is shown. - If the address matches multiple patterns, the balance on all blockchains is shown. (not tested yet, currently supported currencies don't have matching patterns) Tested on a local instance. Someone else should test it before merging.
brickedsalamander added 1 commit 11 months ago
Owner

I will be testing this today

I will be testing this today
brickedsalamander added 1 commit 11 months ago
Owner

Testing right now.

Testing right now.
Poster
Collaborator

Have you tested this for a few hours on your local instance?

Have you tested this for a few hours on your local instance?
Owner

@brickedsalamander

Hey! Sorry for being a little off lately. I've been pretty busy with a lot of stuff.

This next week I will start getting hands on everything again and I will be further testing this and merging it.

Hope you're doing fine!

@brickedsalamander Hey! Sorry for being a little off lately. I've been pretty busy with a lot of stuff. This next week I will start getting hands on everything again and I will be further testing this and merging it. Hope you're doing fine!
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: pluja/iseeyour.cash#54
Loading…
There is no content yet.