Skip to content

feat(dapi): add totalCreditsInPlatform endpoint #2051

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

Closed
wants to merge 35 commits into from
Closed

feat(dapi): add totalCreditsInPlatform endpoint #2051

wants to merge 35 commits into from

Conversation

owl352
Copy link
Contributor

@owl352 owl352 commented Aug 13, 2024

Issue being fixed or feature implemented

Currently the getTotalCredits method cannot be called in js-dapi-client, moreover, even if it is prescribed, the implementation of the method call inside the module is not complete

What was done?

The method has been added to PlatformPromiseClient and PlatformMethodsFacade. A number of changes were also made to the function names

How Has This Been Tested?

Manual testing was performed on a local node built from source code

Breaking Changes

No breaking changes

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 added "!" to the title and described breaking changes in the corresponding section if my code contains any
  • I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@pshenmic pshenmic assigned owl352 and unassigned owl352 Aug 14, 2024
@pshenmic pshenmic added this to the v1.1.0 milestone Aug 14, 2024
@owl352 owl352 marked this pull request as ready for review August 14, 2024 08:49
@shumkov shumkov changed the title fix(platform)!: fixes for correct work totalCreditsOnPlatformQuery on nodejs feat(dapi): add totalCreditsInPlatform endpoint Aug 14, 2024
@shumkov shumkov changed the title feat(dapi): add totalCreditsInPlatform endpoint feat(dapi): add totalCreditsInPlatform endpoint Aug 14, 2024
@shumkov
Copy link
Member

shumkov commented Aug 14, 2024

@owl352 Thank you for your PR. Please fix conflicts and then we can run CI

owl352 added 2 commits August 14, 2024 16:24
…sOnPlatformQuery

# Conflicts:
#	packages/js-dapi-client/lib/methods/platform/getTotalCreditsInPlatform/GetTotalCreditsInPlatformResponse.js
Copy link
Collaborator

@pshenmic pshenmic left a comment

Choose a reason for hiding this comment

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

Looks good to me

@pshenmic
Copy link
Collaborator

Closing in favor of #2059

@pshenmic pshenmic closed this Aug 14, 2024
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.

4 participants