-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[New architecture] Upgrade react-native-svg
#35999
Comments
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as resolved.
This comment was marked as resolved.
Hey! I'm Bartosz from Software Mansion and I would like to work on this task. |
Not a new lib, just an upgrade. Recategorizing... |
Triggered auto assignment to @dylanexpensify ( |
Triggered auto assignment to @tgolen, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
@dylanexpensify Can you please assign this issue to me as I helped with the PR review? |
done! thanks Matt! |
Thanks @muttmuure @dylanexpensify |
@mananjadhav should be paid the standard rate of $500 for this PR review assuming no regressions |
@dylanexpensify This was deployed on production last week and should be ready for payout. Can you please post a payout summary? This was a lib upgrade and hence checklist is not applicable. |
Sure can! |
Payment summary:
|
$500 approved for @mananjadhav based on summary above. |
We need to upgrade
react-native-svg
in order to use the new architecture in the App.More context: https://expensify.slack.com/archives/C01GTK53T8Q/p1707293299589539
Discussion here: https://expensify.slack.com/archives/C01GTK53T8Q/p1707293558602069
The text was updated successfully, but these errors were encountered: