-
Notifications
You must be signed in to change notification settings - Fork 30
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
Havre loic/first time user interests next page 965 #991
Havre loic/first time user interests next page 965 #991
Conversation
…nchers/dev-launchers-platform into havre-loic/first-time-user-interests-next-page-965
…d add context API structure for the UserOnboardingPage and equally add context provider to pages/_app.js
|
@@ -13,6 +13,7 @@ import { ThemeProvider } from 'styled-components'; | |||
import theme from '../styles/theme'; | |||
import Script from 'next/script'; | |||
import iubendaScript from '../scripts/iubendaScript'; | |||
import { OnboardingProvider } from '../../website/src/context/OnboardingContext' |
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 will double-check to see if this is the right _app.js file to import context api
I've had a look at the code, and the code looks fine. I will merge changes and test it out locally as well. There may be two changes I may need to make after merging:
I will create a new branch and create a new PR for upcoming changes. |
No description provided.