Skip to content
This repository has been archived by the owner on Oct 17, 2023. It is now read-only.

Update latest version #3

Merged
merged 14 commits into from
Apr 21, 2023
Merged

Update latest version #3

merged 14 commits into from
Apr 21, 2023

Conversation

tienifr
Copy link

@tienifr tienifr commented Apr 4, 2023

Details

This update is requested in this issue. This PR only merge the latest version 2.5.1 from the original react-native-google-places-autocomplete library.

Fixed Issues

$ Expensify/App#16448
PROPOSAL: Expensify/App#16448 (comment)

Tests

  1. Update commit hash e8346780c43c33b55b62d0241d8a209b48c522a5 for react-native-google-places-autocomplete in package.json then run npm install
  2. Open the app
  3. Open Workspaces > connect bank account
  4. Click on connect manually
  5. Fill in step 1 information (eg data: routing number = 210021002, acc no = 1234)
  6. In the company address, type in address but don't select any option
  7. Click outside (ex: click on the city field)
  8. Verify that the suggestion text below the company address is displayed
  • Verify that no errors appear in the JS console

Offline tests

QA Steps

  1. Update commit hash e8346780c43c33b55b62d0241d8a209b48c522a5 for react-native-google-places-autocomplete in package.json then run npm install
  2. Open the app
  3. Open Workspaces > connect bank account
  4. Click on connect manually
  5. Fill in step 1 information (eg data: routing number = 210021002, acc no = 1234)
  6. In the company address, type in address but don't select any option
  7. Click outside (ex: click on the city field)
  8. Verify that the suggestion text below the company address is displayed
  • Verify that no errors appear in the JS console

Screenshots/Videos

Web
Screen.Recording.2023-04-04.at.16.10.04.mov
Mobile Web - Chrome
Screen.Recording.2023-04-04.at.16.11.12.mov
Mobile Web - Safari
Screen.Recording.2023-04-04.at.16.10.39.mov
Desktop
Screen.Recording.2023-04-04.at.16.14.12.mov
IOS
Screen.Recording.2023-04-04.at.21.37.34.mov
Android
Screen-Recording-2023-04-04-at-21.34.26.mp4

@tienifr

This comment was marked as outdated.

@iwiznia
Copy link

iwiznia commented Apr 10, 2023

image

Can't merge this... should we remove this requirement from this repo?

@eVoloshchak
Copy link

Can't merge this... should we remove this requirement from this repo?

It seems logical to remove it for the forked repositories, but if that's a security concern - we can just cherry-pick those changes and rebase this PR with a signed commit

@iwiznia
Copy link

iwiznia commented Apr 10, 2023

@tienifr
Copy link
Author

tienifr commented Apr 21, 2023

Bump @eVoloshchak @iwiznia to check if the internal discussion had any updates.

@eVoloshchak
Copy link

Looks like it was decided to remove the signed commits requirement in all of the forks.
This is supposed to be done in https://github.com/Expensify/Expensify/issues/275205, but I don't have access to this issue, can't verify.
@iwiznia, could you please check if there are any updates on this?

@iwiznia
Copy link

iwiznia commented Apr 21, 2023

Oh sorry, yes, it is done. Merging.

@iwiznia iwiznia merged commit 6f436a0 into Expensify:master Apr 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants