Skip to content
This repository was archived by the owner on Apr 14, 2024. It is now read-only.

Add login and blacklist#3

Open
TachisAlopex wants to merge 2 commits intoPawCorp:masterfrom
TachisAlopex:add-login-and-blacklist
Open

Add login and blacklist#3
TachisAlopex wants to merge 2 commits intoPawCorp:masterfrom
TachisAlopex:add-login-and-blacklist

Conversation

@TachisAlopex
Copy link

@TachisAlopex TachisAlopex commented Aug 3, 2022

Hello! I would like to add two features to the e621 getter, login and blacklist:
image
Login will allow the user to use the votedup:me tag (and voted/voteddown...), as well as being able to use private sets and load the user's blacklist.

When you check the login, it will prompt for credentials:
image

After filling it out and clicking "Save credentials" it authenticates and persists your login information (will save to localStorage, but will not be exported in the save/load control
image

If your information is wrong, it will show an error string that will go away after editing either field:
image

When you put in this information, it will automatically fetch and fill out the blacklist:
image

You can modify the blacklist (or turn it off altogether). If you didn't login, you can still manually use the blacklist. It defaults to an empty textbox (which won't filter anything)
image

So what do you think? I tried to stay consistent with the rest of the style of the code, but it's my first time working in React, so let me know if anything is off. Thanks!

Edit: slightly changed the wording, and this time made sure my code was run through the linter. Thanks!

This also will pull the blacklist from the user's account automatically if logged in.
The user can disable or edit the list if they wish.
@TachisAlopex TachisAlopex force-pushed the add-login-and-blacklist branch from 4d37626 to c54de55 Compare August 14, 2022 03:23
@pupgray pupgray self-assigned this Aug 27, 2022
@pupgray
Copy link
Collaborator

pupgray commented Aug 27, 2022

Hi thanks so much for the contribution! I take august for vacations, so I'll look more closely at this in a bit!

Thank you so much <3

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants