-
Notifications
You must be signed in to change notification settings - Fork 53
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
add deprecation modal #4296
add deprecation modal #4296
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Skipped Deployments
|
@decentralauren Should the 'why' this modal show not show before directing a user to a link? (copy order) |
@@ -40,6 +42,20 @@ export const resetLastSplash = async () => { | |||
const Splash = ({ animation, isLoggedIn }) => { | |||
const [checked, setChecked] = useState(false) | |||
const [shouldAnimate, setShouldAnimate] = useState(isLoggedIn !== true || isMobileNative) | |||
const { showDeprecationDialog } = useDeprecationDialog() | |||
|
|||
useEffect(() => { |
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.
limit the amount of times you display it or some kind of exponential backoff
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.
added tho following backoff:
- shows first 3 times
- gradually lowers the probability of it showing
- at some point reset.
Good enough?
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.
add: gradually show the modal less
Description
This adds a modal with a notice to UK users.
About # (link your issue here)
#4292
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist: