Skip to content
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-document-picker #35991

Closed
j-piasecki opened this issue Feb 7, 2024 · 17 comments
Closed

[New architecture] Upgrade react-native-document-picker #35991

j-piasecki opened this issue Feb 7, 2024 · 17 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 NewFeature Something to build that is a new item. Reviewing Has a PR in review

Comments

@j-piasecki
Copy link
Contributor

We need to upgrade react-native-document-picker 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/p1707293474881669

@j-piasecki j-piasecki added AutoAssignerAppLibraryReview Auto assign someone to review a new library being added to App Weekly KSv2 labels Feb 7, 2024

This comment was marked as outdated.

This comment was marked as resolved.

@roryabraham
Copy link
Contributor

Just an upgrade, not a new lib. Recategorizing...

@roryabraham roryabraham added NewFeature Something to build that is a new item. and removed AutoAssignerAppLibraryReview Auto assign someone to review a new library being added to App labels Feb 7, 2024

This comment was marked as outdated.

@WojtekBoman
Copy link
Contributor

WojtekBoman commented Feb 8, 2024

Hey! I’m Wojtek from Software Mansion, an expert agency, and I’d like to work on this issue.

Copy link

melvin-bot bot commented Feb 12, 2024

Triggered auto assignment to @yuwenmemon, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@hoangzinh
Copy link
Contributor

@yuwenmemon could you help to assign me as a C+ and add Bug/Feature label so Melvin will assign a BZ member? Thanks

@yuwenmemon
Copy link
Contributor

Of course!

@hoangzinh hoangzinh moved this from Todo to In Progress in RN: New Architecture Feb 13, 2024
@muttmuure muttmuure added the Bug Something is broken. Auto assigns a BugZero manager. label Feb 13, 2024
Copy link

melvin-bot bot commented Feb 13, 2024

Triggered auto assignment to @adelekennedy (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Feb 13, 2024
@muttmuure
Copy link
Contributor

Adding BZ for C+ payment

@roryabraham
Copy link
Contributor

C+ payment should be issued at standard rate of $500 assuming no regressions

Copy link

melvin-bot bot commented Feb 21, 2024

@yuwenmemon, @hoangzinh, @j-piasecki, @WojtekBoman, @adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!

Copy link

melvin-bot bot commented Feb 23, 2024

@yuwenmemon, @hoangzinh, @j-piasecki, @WojtekBoman, @adelekennedy Huh... This is 4 days overdue. Who can take care of this?

@hoangzinh
Copy link
Contributor

@adelekennedy the PR has been deployed to Prod since 15/2. #36118 (comment)
Melvin didn't update the payment status here. Could you help to process payment? Thanks

@adelekennedy
Copy link

@hoangzinh thank you! I think the only payment due here is for C+ (@yuwenmemon or @hoangzinh let me know if I'm missing something)

Offer sent to you @hoangzinh

@hoangzinh
Copy link
Contributor

Accepted. Thanks @adelekennedy

@adelekennedy
Copy link

contract ended and payment remitted - thanks @hoangzinh!

@roryabraham roryabraham moved this from In Progress to Done in RN: New Architecture Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 NewFeature Something to build that is a new item. Reviewing Has a PR in review
Projects
No open projects
Development

No branches or pull requests

9 participants