When adding new dependencies, we use fixed versions.
Don't commit package-lock.json
if you're not making any changes to the libraries.
If the issue you're working on requires changes in both API and Frontend, give your Git branches the same name. CI will automatically pull the correct API's branch when testing the Frontend.
We love screenshots - or better: screencasts. Include some in your pull requests to demonstrate your changes and you will have our eternal gratitude.
I18n
The strings must be internationalized. See /help/developers/translations.
Update the language files npm run build:langs
and commit them to reflect the changes.
Librairies
Whenever it's possible, we must use styled-components
to write styles. See OC Styleguide.
We're getting rid of Bootstrap
and material-ui
. Please don't use them for new developments.
Icons must be imported from the styled-icons library.
Testing
Tests written with Cypress must follow our good practices conventions.