-
Notifications
You must be signed in to change notification settings - Fork 4
Add landing page before token exchange #1255
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
@@ -114,7 +114,7 @@ router.post( | |||
) | |||
|
|||
router.get( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we redirect from old URL to prevent links from becoming 404?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm not sure what you mean, the "old URL" is now the landing page
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When we deploy it, would existing links from Hope or hedvig-com break?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't see how anything would break 🤔
From HOPE users will just land on this page instead of being redirected automatically.
It doesn't implicate hedvig-com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
What?
Add a landing page before we do the token exchange.
Why?
There has been issues with links that become invalid after being visited multiple times.
With this solution we can send out the link to the landing page, and the user can then click on the button to get to the token exchange.
Ticket(s): []