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

[No QA] Add documentation about error onyx best practice under API.md #9629

Merged
merged 4 commits into from
Jun 30, 2022

Conversation

yuwenmemon
Copy link
Contributor

@yuwenmemon yuwenmemon commented Jun 29, 2022

Pullerbearing (@neil-marcellini)
cc @nkuoch @marcaaron @tgolen @iwiznia @Julesssss @MariaHCD @aldo-expensify @luacmartins

Details

Adding what was discussed in this thread about errors returned and stored in Onyx being stored under error

Fixed Issues

None

Tests

None

@yuwenmemon yuwenmemon self-assigned this Jun 29, 2022
@yuwenmemon yuwenmemon requested a review from a team as a code owner June 29, 2022 18:01
@melvin-bot melvin-bot bot requested review from neil-marcellini and removed request for a team June 29, 2022 18:01
luacmartins
luacmartins previously approved these changes Jun 29, 2022
docs/API.md Outdated Show resolved Hide resolved
Copy link
Contributor

@neil-marcellini neil-marcellini left a comment

Choose a reason for hiding this comment

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

Nice, simple and to the point. I agree with Aldo's suggestion.

iwiznia
iwiznia previously approved these changes Jun 29, 2022
@yuwenmemon yuwenmemon dismissed stale reviews from iwiznia and luacmartins via 9e47f26 June 29, 2022 20:05
Co-authored-by: Aldo Canepa Garay <87341702+aldo-expensify@users.noreply.github.com>
aldo-expensify
aldo-expensify previously approved these changes Jun 29, 2022
iwiznia
iwiznia previously approved these changes Jun 29, 2022
luacmartins
luacmartins previously approved these changes Jun 29, 2022
docs/API.md Outdated Show resolved Hide resolved
docs/API.md Outdated Show resolved Hide resolved
Co-authored-by: Robert Chen <robert@expensify.com>
@yuwenmemon yuwenmemon dismissed stale reviews from luacmartins, iwiznia, and aldo-expensify via cc49970 June 30, 2022 05:22
Co-authored-by: Maria D'Costa <maria@expensify.com>
Copy link
Contributor

@Julesssss Julesssss left a comment

Choose a reason for hiding this comment

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

Thanks for adding, looks good

@yuwenmemon yuwenmemon merged commit 6ec9969 into main Jun 30, 2022
@yuwenmemon yuwenmemon deleted the yuwen-errorMessage branch June 30, 2022 16:26
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

OSBotify commented Jul 8, 2022

🚀 Deployed to staging by @yuwenmemon in version: 1.1.81-0 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 failure ❌

@roryabraham roryabraham changed the title Add documentation about error onyx best practice under API.md [No QA] Add documentation about error onyx best practice under API.md Jul 8, 2022
@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.1.82-5 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants