@simonrabourg-wq
SimonR
- Merged PRs
- 36
- Reviews
- 131
- Repos
- 3
- Followers
- 0
- Merged
- 90%
tomorro/aura#422we should have a way to activate this globally only on testing environment.APR 2026tomorro/aura#422fixedAPR 2026tomorro/aura#422won't happen as these properties are not suppose to change after a failureAPR 2026tomorro/aura#422Nope, already had been fixed with citation.risk === 'UNSURE'APR 2026tomorro/aura#422could be but it is ok like thisAPR 2026tomorro/aura#422true but this check is only retriggered on property change via useEffect so what you describe won't happenAPR 2026tomorro/aura#422true, bug bot found it.APR 2026tomorro/aura#422not relevantAPR 2026tomorro/aura#421global work on the unit tests to be done.APR 2026tomorro/aura#421Good note but review is not linked to legal analysis for now on db, only the other way aroundAPR 2026tomorro/aura#431not sure to understand why aggregating both?APR 2026tomorro/aura#418different localization in the file, are we sure?APR 2026tomorro/aura#418could we externalize that in private rendering function?APR 2026tomorro/aura#415would worth to forbid auto-demote (via api only)APR 2026tomorro/aura#415Dialog could be extracted elsewhere isn't it?APR 2026tomorro/aura#415env var?APR 2026tomorro/aura#415specific component?APR 2026tomorro/aura#416those blocks could be factorized within a same private functionAPR 2026tomorro/aura#402you're right for this layer it is enough.APR 2026tomorro/aura#398This one could be updated and improved, but it implies a structure change on the backend and I'm not as confident to deliver it now. I propose to merge it like this and arrange thiAPR 2026