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

[Custom Fields] Screen on the example app to test the feature #3084

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

hichamboushaba
Copy link
Member

@hichamboushaba hichamboushaba commented Aug 16, 2024

This PR just adds a screen to the example app to test the changes added in #3082. The PR seems big, but since the code all goes to the example app, I don't think you need to spend too much time reviewing the code.

I opted to add Compose to the example app, if you have any concerns please share them.

Steps

  1. Open the example app.
  2. Sign in.
  3. Tap on Woo, then select a store.
  4. Tap on Orders.
  5. Tap on "View Order Custom Fields"
  6. Enter the ID of an existing order.
  7. On the next screen, confirm the screen loads the custom fields correctly.
  8. Make some changes.
  9. Save.
  10. Confirm the flow works as expected.
  11. Repeat 4-10 for Products.

Copy link
Contributor

@JorgeMucientes JorgeMucientes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one @hichamboushaba 👏🏼
It's good to see Compose code in the FluxC example app, thanks for adding it! I was able to test custom fields editing easily with the UI you've added and everything worked well!

:shipit:

@hichamboushaba hichamboushaba merged commit 6ab361f into woo/meta-data-store Aug 22, 2024
2 of 12 checks passed
@hichamboushaba hichamboushaba deleted the woo/meta-data-store-2 branch August 22, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants