Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

feat: comprehensive error codes #262

Merged
merged 5 commits into from
Sep 15, 2021
Merged

Conversation

shumkov
Copy link
Member

@shumkov shumkov commented Sep 8, 2021

Issue being fixed or feature implemented

Provide comprehensive error codes for DAPI responses

What was done?

  • Updated DPP
  • Updated DAPI Client

How Has This Been Tested?

With tests

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shumkov shumkov changed the title feat: introduce comprehe error codes feat: introduce compreheni error codes Sep 8, 2021
@shumkov shumkov changed the title feat: introduce compreheni error codes feat: comprehensive error codes Sep 8, 2021
@shumkov shumkov added this to the v0.21.0 milestone Sep 8, 2021
@shuplenkov shuplenkov merged commit baced33 into v0.21-dev Sep 15, 2021
@shuplenkov shuplenkov deleted the update-dapi-client-dpp-wallet branch September 15, 2021 07:54
@shuplenkov shuplenkov restored the update-dapi-client-dpp-wallet branch September 15, 2021 07:54
antouhou pushed a commit that referenced this pull request Oct 25, 2021
Co-authored-by: Konstantin Shuplenkov <konstantin.shuplenkov@dash.org>
antouhou pushed a commit that referenced this pull request Oct 25, 2021
Co-authored-by: Konstantin Shuplenkov <konstantin.shuplenkov@dash.org>
antouhou pushed a commit that referenced this pull request Oct 26, 2021
Co-authored-by: Konstantin Shuplenkov <konstantin.shuplenkov@dash.org>
@shumkov shumkov deleted the update-dapi-client-dpp-wallet branch October 26, 2021 15:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants