César Vargas Casaseca

Results 13 comments of César Vargas Casaseca

The action looks good to me! I just added a couple of small code remarks. :) Besides that, it seems to me that the implemented layout does not fully match...

Done with https://github.com/woocommerce/woocommerce-ios/pull/7450

Thanks for finding that issue @jostnes! I created an [issue](https://github.com/woocommerce/woocommerce-ios/issues/7470) and will address it right away :)

@joshheald I checked this issue, and I think it was fixed after making the view scrollable in the context of https://github.com/woocommerce/woocommerce-ios/issues/7286: the user can scroll to reach the confirm button....

Videos for reference from Android https://github.com/woocommerce/woocommerce-android/issues/7024 https://user-images.githubusercontent.com/4923871/182574911-b364205f-e04d-4545-a6b1-fc6b6499b8c5.mp4 https://user-images.githubusercontent.com/4923871/182574910-5d8a2bf3-abc3-4140-a734-af50885443b9.mp4 https://user-images.githubusercontent.com/4923871/182574909-b7d31cde-fcb7-48d7-9436-43461ba74b3d.mp4

Notice should be permanent with this design: ![image](https://user-images.githubusercontent.com/1864060/184328972-1ed197b0-f5f8-4ffe-85be-f406269fa08e.png)

hi @Ecarrion Thanks for checking that :) I updated the PR description with the new format for the links, as we agreed to add the `/mobile/ `segment so redirections can...

Thanks for trying again @Ecarrion :) There are two points you can do to make sure you have the right configuration in your phone: - Be sure you run the...

Thanks for checking that again @Ecarrion :) > The problem was on my end, when I updated the links to the add mobile path, the string visible to me changed,...