-
Notifications
You must be signed in to change notification settings - Fork 2
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
Security issues + i18n refactors #91
Conversation
@p3rcypj Can you please review that the branch is working for you? I've updated some packages to keep the security messages to a minimum. I've also enabled linting messages in the "yarn start" console and hidden them in the browser. That's what we are used to in CRA projects and I think it's a sensible configuration. |
Snyk current status:
d2 and @dhis2/d2-i18n packages are already in the last version, we'd need upstream to update them. |
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.
Everything working as a charm! 🌟
|
📌 References
📝 Implementation
📹 Screenshots/Screen capture
🔥 Notes to the tester