(PC-15161) build(deps): bump react-native to 0.68.2 - #2948
Conversation
anoukhello
changed the title
anoukhello
changed the title
anoukhello
changed the title
anoukhello
changed the title
anoukhello
changed the title
a038fd4 to
9939e28
Compare
yorickeando
left a comment
yorickeando
left a comment
There was a problem hiding this comment.
Trop bien, super découpage très clair. Je fais confiance à tes différents builds iOS et Android qui marchent. Tu as pu tester sur deux versions d'Android différentes, pour être un peu plus confiante ?
Il y a un petit code smell gérable (autre que FIXME).
| source "https://rubygems.org" | ||
|
|
||
| # You may use http://rbenv.org/ or https://rvm.io/ to install and use this version | ||
| ruby '2.7.4' |
There was a problem hiding this comment.
@yorickeando c'est plutôt ça qui permet d'imposer la version de ruby. Le fichier .ruby-version permet de dire à ton gestionnaire de version de ruby (rbenv, rvm, etc) d'utiliser cette version spécifiquement pour ce projet
kopax-polyconseil
left a comment
kopax-polyconseil
left a comment
There was a problem hiding this comment.
Un grand merci pour cette PR de maj. 🎉
J'insisterai sur l'importance des tests de non regression. Si on supprime TMX, il y a aussi une route sur le nextBeneficiaryTest qui lancait un profiling "on-demand", p-e à retirer si ce n'est pas déjà fait.
| > | ||
| <View | ||
| accessible={true} | ||
| collapsable={false} |
There was a problem hiding this comment.
ça peut valoir le coup de regarder dans la version web si c'est props apparraissent maintenant dans le dom (en [Object object]) et auquel cas voir si une mise à jour de react-native-web est possible pour retirer ces props, sinon créer un ticket sur leur repo.
| import { useKeyboardAdjust } from 'ui/components/keyboard/useKeyboardAdjust' | ||
|
|
||
| export const Favorites: React.FC = () => { | ||
| useKeyboardAdjust() |
There was a problem hiding this comment.
il me semble que ceci réglait l'ouverture du clavier en redimensionnant certaines view, et ça parraissait utile (du moins en avril 2021), es-tu sur que ça ne casse rien ?
There was a problem hiding this comment.
Sur les favoris on n'ouvre pas le clavier il me semble, donc l'utilisation de ce hook ici n'est pas adapté
| import { Form } from 'ui/web/form/Form' | ||
|
|
||
| export function Search() { | ||
| useKeyboardAdjust() |
There was a problem hiding this comment.
je pense notamment au page comme SetEmail, etc... il faut vraiment retest cette partie.
There was a problem hiding this comment.
On n'utilise pas ce hook sur le SetEmail ou autre page du parcours d'inscription. D'après le screenshot en commentaire de la PR, le comportement est le même avec ou sans la lib sur la page de recherche
…roj after pods installation
|
Kudos, SonarCloud Quality Gate passed! |
* (PC-15161) build(deps): bump react-native to 0.68.2 * (PC-15161) build(deps): add .ruby-version file in order to install ios pods * (PC-15161) build(deps): update Gemfile.lock/Podfile.lock/project.pbxproj after pods installation * (PC-15161) build(deps): add babel plugin polyfills for tests * (PC-15161) build(deps): remove unnecessary and unmaintained package * (PC-15161) build(deps): update ios/PassCulture/AppDelegate.m to make ios build work * (PC-15161) remove user-profiling from phone validation * (PC-15161) build(deps): remove react-native-user-profiling
4 participants
Ticket jira: https://passculture.atlassian.net/browse/PC-15161
Changelog: https://github.com/facebook/react-native/releases/tag/v0.68.2
Doc suivie pour la màj:
Je voulais l'ignorer (grâce à
LogBox.ignoreLogsmais ça ne semble pas fonctionner) jusqu'à ce que la lib react-native-dash retire ses références àViewPropTypes(obipawan/react-native-dash#36). Cependant la lib n'a plus vraiment l'air d'être maintenue, je propose donc de supprimer son utilisation => ticket JIRACertains de ces warnings sont aussi causés par
react-native-date-pickerqu'il faudrait mettre à jour en version 4.2.2 => ticket JIRA