Skip to content
This repository has been archived by the owner on Feb 16, 2024. It is now read-only.

INT-1898 Use custom notifs for auth flow #825

Merged
merged 3 commits into from
Sep 20, 2023
Merged

INT-1898 Use custom notifs for auth flow #825

merged 3 commits into from
Sep 20, 2023

Conversation

hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Sep 20, 2023

@ghost
Copy link

ghost commented Sep 20, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@@ -18,6 +20,17 @@ import { ToastContainer } from 'react-toastify';
import 'react-toastify/dist/ReactToastify.css';
import { useTheme } from '../shared/Snippet/useTheme';

const ToastContainerProps = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you rename it to toastContainerProps (lowercase)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addressed, thanks

@hbjORbj hbjORbj merged commit 65703fd into main Sep 20, 2023
5 checks passed
@hbjORbj hbjORbj deleted the custom-notifs branch September 20, 2023 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants