luludotdev / beatsaver-reloaded

BeatSaver Again
ISC License
181 stars 50 forks source link

Base code for resend validation email #103

Closed revelcw closed 3 years ago

revelcw commented 3 years ago

Proposed Changes

Please describe the changes this PR makes and why it should be merged. If it fixes a bug or resolves a feature request, be sure to reference that issue so it can be closed automatically.

Platforms

This pull request modifies: (select all that apply)

Types of Changes

This pull request contains: (select all that apply)

Checklist

Further Comments

So, after my issue #102 where I couldn't upload because my account was not verified and I didn't get the email (Please check it out), I decided to make a PR with some new code.

I am hitting /auth/resend from a button on the upload page. It gets the user with ctx.state.user and sending a new email to them.

But... THIS IS NOT DONE YET. I couldn't run it because I don't have any of the env keys for testing. So this is only a basic guideline for a resend email implementation. A further note, I don't know exactly how this code was written, so I could only make some small assumptions about how to make this. If this can not be done please delete my account from #102. Thank you, revelcw

luludotdev commented 3 years ago

I am not accepting PRs at this time due to an internal rewrite that is currently in development

revelcw commented 3 years ago

Is BeatSaver down? I can't upload anything. If so, in the meantime could you add a more useful error like Sorry BeatSaver is slow right now. Try again later.