mirror of
https://github.com/TryGhost/Ghost.git
synced 2025-03-25 02:31:59 -05:00
c6621dc17d
refs https://github.com/TryGhost/Team/issues/584 The current support email verification flow uses an API endpoint as verification URL inside the emails. This is a bad pattern, and also has the side effect that it shows a JSON error if something goes wrong. To fix this, this commit updates the whole flow to use the same pattern as newsletters: - You can update the `members_support_address` setting directly via the edit endpoint of settings. - Changes to that (and future 'guarded' email properties) are blocked and generate verification emails automatically. - When an email verification has been sent, the meta property `sent_email_verification` is set. Other changes: - Underlying, the implementation of email verificaton has moved from the (old) members service to the settings BREAD service. This makes it easier to add extra email addresses in settings later on that are not related to 'members'. - Now you can update the `members_support_address` by updating the settings directly, so the `updateMembersEmail` endpoint has been deprecated and is mapped to the new behaviour. - The SingleUseTokenProvider threw a `UnauthorizedError` error if a token was expired or invalid. Those errors are caught by the admin app, and causes it to do a page reload (making the error message and modals invisible). To fix that, I've swapped it with a validation error. Future changes: - Existing emails that have been sent 24h before this change is applied, still use the `validateMembersEmailUpdate` API endpoint. This endpoint has not been removed for now, to not break those emails. In a future release, we should remove this. Changes to admin: https://github.com/TryGhost/Admin/pull/2426 |
||
|---|---|---|
| .. | ||
| __snapshots__ | ||
| actions.test.js | ||
| config.test.js | ||
| custom-theme-settings.test.js | ||
| db.test.js | ||
| email-previews.test.js | ||
| emails.test.js | ||
| explore.test.js | ||
| files.test.js | ||
| images.test.js | ||
| integrations.test.js | ||
| invites.test.js | ||
| key-authentication.test.js | ||
| labels.test.js | ||
| media.test.js | ||
| members-exporter.test.js | ||
| members-importer.test.js | ||
| members-newsletters.test.js | ||
| members-stripe-connect.test.js | ||
| members.test.js | ||
| newsletters.test.js | ||
| notifications.test.js | ||
| oembed.test.js | ||
| offers.test.js | ||
| pages.test.js | ||
| posts.test.js | ||
| redirects.test.js | ||
| roles.test.js | ||
| session.test.js | ||
| settings-files.test.js | ||
| settings.test.js | ||
| site.test.js | ||
| slugs.test.js | ||
| snippets.test.js | ||
| stats.test.js | ||
| tags.test.js | ||
| themes.test.js | ||
| tiers.test.js | ||
| users.test.js | ||
| utils.js | ||
| webhooks.test.js | ||