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

fix: use error.httpStatusCode instead of error.status for d2 errors #35

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

mediremi
Copy link
Contributor

Porting this app to the app platform required updating d2 from v27.0.0-5 to v31.9.0. Between these versions, the way d2 returns API related errors changed - in v27 the http status of an API error is stored in the status field while in v31 it is stored in the httpStatusCode field.

This PR replaces every instance of error.status with error.httpStatusCode.

@mediremi mediremi merged commit b6e9275 into master Mar 29, 2021
@mediremi mediremi deleted the fix/delete-last-key-workspace branch March 29, 2021 09:22
dhis2-bot added a commit that referenced this pull request Mar 29, 2021
## [1.3.2](v1.3.1...v1.3.2) (2021-03-29)

### Bug Fixes

* use error.httpStatusCode instead of error.status for d2 errors ([#35](#35)) ([b6e9275](b6e9275))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 1.3.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants