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

Update Compose #264

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Update Compose #264

merged 1 commit into from
Feb 21, 2017

Conversation

stuartpb
Copy link
Member

No description provided.

@stuartpb
Copy link
Member Author

stuartpb commented Feb 21, 2017

Moves to compose.com via legacies, per ruminations from #43.

Tried to register with https://mailsac.com/inbox/compose-2017-02-21@tmail.opws.org but it requires a credit card and I CBA to fake payment credentials so I skipped out on that. Rest of these notes / profile changes are based on experiments via my own account

Made me update my payment info because my old info was expired

Stripe sent me a verification SMS, then couldn't use my stored CC so I still had to re-up manually

destination is dashboard if resetting password while logged in, login if not (#68) - the flash says "please log in" either way

requesting a second password reset token after the first doesn't send it, maybe because the old one still works, even after you use it

Wait, no, it was sent, it just didn't arrive until it was eleven damn minutes late, at which point it had already expired (I think these have a fifteen-minute lifetime? that's an estimate based on the timestamp and this email being broken now) and I'd already determined that these have an expiration problem

Passwords above 999 don't reject so I've removed the max: it does appear that they're being truncated, though (#233), so maybe that can be tested and profiled in the future.

@stuartpb stuartpb merged commit 7a9a4b4 into master Feb 21, 2017
@stuartpb stuartpb deleted the compose branch February 21, 2017 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant