Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add new block type #247

Merged
merged 3 commits into from
Nov 20, 2020
Merged

Add new block type #247

merged 3 commits into from
Nov 20, 2020

Conversation

rg911
Copy link
Contributor

@rg911 rg911 commented Nov 19, 2020

@rg911 rg911 requested a review from fboucquez November 19, 2020 16:38
$ref: ../../../schemas/UInt32.yml
description: Number of voting eligible accounts.
harvestingEligibleAccountsCount:
type: string # uint64
Copy link
Contributor

Choose a reason for hiding this comment

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

can this be a reference to one of the schemas? Maybe create a UInt64.yml if not using Amount.yml?
This is for the java generation that automatically converts strings to BigInteger

Copy link
Contributor

Choose a reason for hiding this comment

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

@rg911 , pushed this change. PR approved

@fboucquez fboucquez self-requested a review November 20, 2020 16:25
@rg911 rg911 merged commit 562be9b into symbol:main Nov 20, 2020
rg911 added a commit that referenced this pull request Dec 8, 2020
* Updated lock secret endpoints

* Added transactions transferMosaicId filter

* Creating new version 0.10.1

* Added endpoints to get finalization proof at a given height or point

* Point to Epoch and redesigned routes

* Added DTOs

* Made epoch endpoint clearer

* Renamed a field, added some descriptions

* Update spec/core/finalization/schemas/MessageGroup.yml

Co-authored-by: Xavi Artigas <xavierartigas@yahoo.es>

* Update spec/core/finalization/routes/getFinalizationProofAtEpoch.yml

* Fixed missing index entries

* Updated epoch/point types

* Added transfer amount options to filter transfer transactions

* Updated transferMosaicId param description

Co-authored-by: Xavi Artigas <xavierartigas@yahoo.es>

* Typo

* Fixed filter type

* Improved search transactions endpoint description regarding alias resolution and addresses (#231)

Co-authored-by: Xavi Artigas <xavierartigas@yahoo.es>

* Removed id from namespace meta

This field has been moved to the top-level

https://github.com/nemtech/symbol-openapi/blob/main/spec/plugins/namespace/schemas/NamespaceInfoDTO.yml#L3

* unlocke account (#243)

* Added new node public key (#244)

* Merkle endpoints for state proof  (#240)

* POC merkle endpoints
Renamed Status to LockStatus enum

* Updated Merkle Info name and description

* Added missing endpoints

* Added missing params

* Add new block type (#247)

* Add new block type

* Added missing export

* Added Uint64 to simplify generation.

Co-authored-by: fernando <fboucquez@gmail.com>

* Added merkle tree schema (#248)

Co-authored-by: fboucquez <fboucquez@gmail.com>
Co-authored-by: Xavi Artigas <xavierartigas@yahoo.es>

* Added version to states (#252)

* Added missing failure status (#250)

* Added missing failure status

* Added a few more missing statuses

* Moved Roles types to a generic number (#258)

* Updated network types (#257)

* Finality proof fix (#253)

* Finality proof fix

* Updated signatureSchema desc

* Namespace Transaction required fields fix (#242)

On namespace transactions, duration or parent id is required, but not both at the same time. 

If required is in place, user (and generated code) would think that duration and parentid are always provided.

* Updated version and CHANGELOG.md (#254)

* Update CHANGELOG.md (#259)

Co-authored-by: Martin Ayora <martin.ayora@nem.software>
Co-authored-by: Travis CI User <travis@example.org>
Co-authored-by: Roger Hernandez <rogerernandez@hotmail.com>
Co-authored-by: Xavi Artigas <xavierartigas@yahoo.es>
Co-authored-by: Steven Liu <xian.f.liu@gmail.com>
Co-authored-by: Martin Ayora <17176539+martinayora@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants