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

fix: Fix iroha_genesis compilation without transparent_api feature #5056

Conversation

dima74
Copy link
Contributor

@dima74 dima74 commented Sep 10, 2024

Context

Fixes #5049

Solution

  • Moved apply_parameter function to the data_model
  • Also fixed port for some tests:
    • reported_length_should_be_accurate used port 11_170 which conflicts with register_offline_peer
    • json_and_scale_statuses_equality used port 11_200 which conflicts with restarted_peer_should_have_the_same_asset_amount

Checklist

  • I've read CONTRIBUTING.md.
  • (optional) I've written unit tests for the code changes.
  • All review comments have been resolved.
  • All CI checks pass.

mversic
mversic previously approved these changes Sep 11, 2024
0x009922
0x009922 previously approved these changes Sep 11, 2024
nxsaken
nxsaken previously approved these changes Sep 11, 2024
@nxsaken nxsaken force-pushed the diralik/fix-iroha-genesis-transparent-api branch from 166649c to 1accc6b Compare September 11, 2024 07:12
@dima74 dima74 dismissed stale reviews from mversic, 0x009922, and nxsaken via e3c79b7 September 11, 2024 10:55
@dima74 dima74 force-pushed the diralik/fix-iroha-genesis-transparent-api branch from 1accc6b to e3c79b7 Compare September 11, 2024 10:55
Signed-off-by: Dmitry Murzin <diralik@yandex.ru>
Signed-off-by: Dmitry Murzin <diralik@yandex.ru>
@mversic mversic force-pushed the diralik/fix-iroha-genesis-transparent-api branch from e3c79b7 to 9f53082 Compare September 12, 2024 07:53
@mversic mversic merged commit 7b84762 into hyperledger-iroha:main Sep 12, 2024
11 checks passed
@dima74 dima74 deleted the diralik/fix-iroha-genesis-transparent-api branch September 13, 2024 09:05
# 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.

Cannot compile iroha_genesis: uses private fields from iroha_data_model
4 participants