-
-
Notifications
You must be signed in to change notification settings - Fork 2
feat(admin): FuzzySearch artist lookup client, endpoint, and key setting (SONA-156) #425
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
Open
sparkyfen
wants to merge
38
commits into
main
Choose a base branch
from
sparky/sona-156-artist-lookup-key
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
73b8528
feat(lookup): add the FuzzySearch reverse image search client (SONA-156)
sparkyfen f36d77f
feat(lookup): add POST /api/admin/artist-lookup (SONA-156)
sparkyfen 536786f
feat(settings): add the Artist lookup section to Connections (SONA-156)
sparkyfen 3d7f369
docs: name FuzzySearch in the architecture diagram and README (SONA-156)
sparkyfen 9d76e87
fix(lookup): tighten the FuzzySearch client and endpoint (SONA-156)
sparkyfen bca3ef0
fix(settings): make the Artist lookup section reachable and readable …
sparkyfen 02fd958
fix(settings): keep focus and contrast through the lookup key states …
sparkyfen 6df227d
fix(lookup): compare tweets by status id and read host aliases safely…
sparkyfen 1b44be0
fix(lookup): restore the mobile.twitter.com alias and fold tweet perm…
sparkyfen a08c3cf
fix(settings): hold Remove key's hover contrast and pin the pending c…
sparkyfen 13fa96b
fix(settings): hold the lookup block steady when the confirmation ope…
sparkyfen 6a07c13
test(settings): pin the cancel() double-submit guard and the live reg…
sparkyfen e2e3a6e
test(settings): pin the lookup confirmation's layout and guards by co…
sparkyfen 931b1aa
fix(lookup): report a rejected stored fetch and fold more source URLs…
sparkyfen e6b09b7
fix(image-proxy): pass through only the stored raster types (SONA-156)
sparkyfen bb47b6d
refactor(lookup): fold the rejected fetch into the proxy and share on…
sparkyfen bf3f2f2
fix(con-card): fall back when the avatar proxy answers a non-image ty…
sparkyfen aba70b3
test(con-card): pin the avatar proxy type guard at its call site (SON…
sparkyfen 1365544
fix(artist-lookup): treat 403 as a refused key and gate uploads on ra…
sparkyfen b721d12
fix(artist-lookup): clear the refusal marker per key source and separ…
sparkyfen 3b04fdf
test(e2e): make the artist-lookup key save step survive a pre-hydrati…
sparkyfen 79f6d49
test(e2e): start each artist-lookup key save attempt from a fresh set…
sparkyfen 8608dcb
fix(fuzzysearch): fold Weasyl title slugs and pin the timeout signal …
sparkyfen 423286d
fix(settings): ignore a reflex second click on the artist-lookup remo…
sparkyfen 227bbd2
fix(fuzzysearch): fold Weasyl user permalinks and compare their paths…
sparkyfen fadd84a
fix(settings): scope the remove-confirmation reflex guard to pointer …
sparkyfen 559c9b2
fix(settings): only swallow a reflex click near where Remove key was …
sparkyfen e4aebf3
fix(settings): widen the reflex box for coarse pointers (SONA-156)
sparkyfen 237afc0
test(e2e): note that the artist-lookup key spec cannot be stress-run …
sparkyfen 207499a
fix(artist-lookup): bound the proxy header wait, cancel failed upstre…
sparkyfen 5218219
fix(fuzzysearch): bound how much of a FuzzySearch response is read (S…
sparkyfen b7da9c4
test(artist-lookup): re-arm the settings spy before each case (SONA-156)
sparkyfen b2b6ead
test(fuzzysearch): give the bounded-read chunk allowance room for str…
sparkyfen ea3e8fa
fix(fuzzysearch): never follow a redirect with the API key, and pin t…
sparkyfen f65010f
fix(artist-lookup): answer a refused key with 424 so it stays out of …
sparkyfen 11ca467
test(theme-contrast): measure the lookup remove hover against the rul…
sparkyfen f20ad6b
fix(artist-lookup): sniff stored image bytes before sending them to F…
sparkyfen 4f47d71
fix(settings): keep the stored key's mask off the page while a deploy…
sparkyfen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.