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.

Sign up for GitHub

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

Jump to bottom

feat: 59 user story client implementing handler for create client #179

Merged
merged 15 commits into from
Apr 1, 2023
Merged

feat: 59 user story client implementing handler for create client #179

merged 15 commits into from
Apr 1, 2023

Conversation

shreyasbhat0
Copy link
Collaborator

Description:

Commit Message

feat: implement handler for create client

see the guidelines for commit messages.

Checklist:

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@shreyasbhat0 shreyasbhat0 self-assigned this Mar 30, 2023
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@902c707). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #179   +/-   ##
=======================================
  Coverage        ?   50.11%           
=======================================
  Files           ?       64           
  Lines           ?     5499           
  Branches        ?        0           
=======================================
  Hits            ?     2756           
  Misses          ?     2743           
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@MuhammedIrfan MuhammedIrfan merged commit 38180b5 into main Apr 1, 2023
@MuhammedIrfan MuhammedIrfan deleted the 59-user-story-client-implementing-handler-for-create-client branch April 1, 2023 07:22
ibrizsabin pushed a commit that referenced this pull request Apr 3, 2023
@shreyasbhat0 @MuhammedIrfan @sabinchitrakar
feat: implement handler for create client (#179)
69d4e34 
* chore: implement register client and client identifier

* chore: implement create client handler

* chore: update core storage

* chore: implement clientstate and consensus state

* chore: implement default for types

* chore: update cargo

* chore: implement consensus state object

* chore: add create client test

* chore: update consensus state decoding

* chore: update message structure for lightclient execution

* chore: update test cases

---------

Co-authored-by: Muhammed Irfan <50794925+MuhammedIrfan@users.noreply.github.com>
ibrizsabin pushed a commit that referenced this pull request Feb 23, 2024
@gcranju @AntonAndell
@DeepakBomjan @izyak @redlarva
feat: persistent messaging added in solidity (#232)
cc29590 
* feat: xcall upgradeable (#179)

* feat: xcall upgradeable

* fix: foundry toml fix

* deploy mock script

* fix: remapping

* fix: env name changed

* fix: script reformed

* fix: upgradeable using openzeppelin

* fix: upgradeable using openzeppelin

* fix: deploy scripts fix

* fix: deploy scripts fix

* forge install: openzeppelin-foundry-upgrades

* fix: Make simple dapp to use same logic as dapp multi (#195)

* fix: adapters deployment scripts added

* fix: adapters included in script

---------

Co-authored-by: AntonAndell <anton@andell.eu>

* fix: network address validation fixed (#212)

* fix: not authorized issue by making handleMessage public (#202)

* fix: not authorized issue by making handleMessage public

* feat: handleBTPMessage tests added

* fix: add fixes for handle Error

* fix: code order optmization in handle message

Signed-off-by: gcranju <qa5505815@gmail.com>

* fix:revert message added in handleRequest

* fix: versions openzeppelin fix

* fix: foundry upgrades deleted

---------

Signed-off-by: gcranju <qa5505815@gmail.com>

* ci: add job to update CHANGELOG.md (#207)

* fix: added rpsc in foundry.toml

* fix: fix env.example

* fix: new message type added in solidity

* feat: add centralized connections in solidity and java (#196)

* feat: solidity xcall connection added

* feat: icon xcall connection added

* fix: checkfee added

* feat: relayer address in centralized connection

* feat: added build scripts

* feat: solidity tests for centralized connection added

* feat: centralized connection name changed for icon

* fix: review comments addressed for solidity part

* fix: variable admin used for relayer/admin

* fix: review comments addressed for javascore part

* fix: variables in build.gradle in centralized cnnection

* fix: java score constructor

* fix: removed void from constructor

* fix: type of receipts to boolean

* fix: solitity tests resolved

* fix: logic corrected for duplicate message

* fix: fee check added in connection

* fix: centralized connection test added in solidity

* fix: admin check removed from getReceipts

Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: onlyAdmin function

* fix: architecture updated (#214)

* fix: architecture updated

* fix: use connectio sn

* style: formatting code

* added scripts for centralized connections

* feat: centralized-connection javascore tests added

* feat: clam fees tests added

---------

Co-authored-by: red__larva <red.x.larva@proton.me>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: unnecessary event log removed

* fix: sendBTPMessage renamed to sendToConnection

* fix: unnecessary event log removed

* feat: add centralized connections in solidity and java (#196)

* feat: solidity xcall connection added

* feat: icon xcall connection added

* fix: checkfee added

* feat: relayer address in centralized connection

* feat: added build scripts

* feat: solidity tests for centralized connection added

* feat: centralized connection name changed for icon

* fix: review comments addressed for solidity part

* fix: variable admin used for relayer/admin

* fix: review comments addressed for javascore part

* fix: variables in build.gradle in centralized cnnection

* fix: java score constructor

* fix: removed void from constructor

* fix: type of receipts to boolean

* fix: solitity tests resolved

* fix: logic corrected for duplicate message

* fix: fee check added in connection

* fix: centralized connection test added in solidity

* fix: admin check removed from getReceipts

Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: onlyAdmin function

* fix: architecture updated (#214)

* fix: architecture updated

* fix: use connectio sn

* style: formatting code

* added scripts for centralized connections

* feat: centralized-connection javascore tests added

* feat: clam fees tests added

---------

Co-authored-by: red__larva <red.x.larva@proton.me>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: centralized-connection test fixed

* fix: proxy request structure with data hash added

* fix: getFee method

* fix: message encoding decoding fixed

* fix: logs removed

* feat: persistent messaging added in solidity

* fix: foundry upgradeable version

* fix: review changes on persistent messaging

---------

Signed-off-by: gcranju <qa5505815@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>
Co-authored-by: DeepakBomjan <44976635+DeepakBomjan@users.noreply.github.com>
Co-authored-by: izyak <76203436+izyak@users.noreply.github.com>
Co-authored-by: red__larva <red.x.larva@proton.me>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>
ibrizsabin pushed a commit that referenced this pull request Feb 23, 2024
@gcranju @AntonAndell
@DeepakBomjan @izyak @redlarva
feat: response message solidity (#234)
fa211ca 
* feat: xcall upgradeable (#179)

* feat: xcall upgradeable

* fix: foundry toml fix

* deploy mock script

* fix: remapping

* fix: env name changed

* fix: script reformed

* fix: upgradeable using openzeppelin

* fix: upgradeable using openzeppelin

* fix: deploy scripts fix

* fix: deploy scripts fix

* forge install: openzeppelin-foundry-upgrades

* fix: Make simple dapp to use same logic as dapp multi (#195)

* fix: adapters deployment scripts added

* fix: adapters included in script

---------

Co-authored-by: AntonAndell <anton@andell.eu>

* fix: network address validation fixed (#212)

* fix: not authorized issue by making handleMessage public (#202)

* fix: not authorized issue by making handleMessage public

* feat: handleBTPMessage tests added

* fix: add fixes for handle Error

* fix: code order optmization in handle message

Signed-off-by: gcranju <qa5505815@gmail.com>

* fix:revert message added in handleRequest

* fix: versions openzeppelin fix

* fix: foundry upgrades deleted

---------

Signed-off-by: gcranju <qa5505815@gmail.com>

* ci: add job to update CHANGELOG.md (#207)

* fix: added rpsc in foundry.toml

* fix: fix env.example

* fix: new message type added in solidity

* feat: add centralized connections in solidity and java (#196)

* feat: solidity xcall connection added

* feat: icon xcall connection added

* fix: checkfee added

* feat: relayer address in centralized connection

* feat: added build scripts

* feat: solidity tests for centralized connection added

* feat: centralized connection name changed for icon

* fix: review comments addressed for solidity part

* fix: variable admin used for relayer/admin

* fix: review comments addressed for javascore part

* fix: variables in build.gradle in centralized cnnection

* fix: java score constructor

* fix: removed void from constructor

* fix: type of receipts to boolean

* fix: solitity tests resolved

* fix: logic corrected for duplicate message

* fix: fee check added in connection

* fix: centralized connection test added in solidity

* fix: admin check removed from getReceipts

Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: onlyAdmin function

* fix: architecture updated (#214)

* fix: architecture updated

* fix: use connectio sn

* style: formatting code

* added scripts for centralized connections

* feat: centralized-connection javascore tests added

* feat: clam fees tests added

---------

Co-authored-by: red__larva <red.x.larva@proton.me>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: unnecessary event log removed

* fix: sendBTPMessage renamed to sendToConnection

* fix: unnecessary event log removed

* feat: add centralized connections in solidity and java (#196)

* feat: solidity xcall connection added

* feat: icon xcall connection added

* fix: checkfee added

* feat: relayer address in centralized connection

* feat: added build scripts

* feat: solidity tests for centralized connection added

* feat: centralized connection name changed for icon

* fix: review comments addressed for solidity part

* fix: variable admin used for relayer/admin

* fix: review comments addressed for javascore part

* fix: variables in build.gradle in centralized cnnection

* fix: java score constructor

* fix: removed void from constructor

* fix: type of receipts to boolean

* fix: solitity tests resolved

* fix: logic corrected for duplicate message

* fix: fee check added in connection

* fix: centralized connection test added in solidity

* fix: admin check removed from getReceipts

Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: onlyAdmin function

* fix: architecture updated (#214)

* fix: architecture updated

* fix: use connectio sn

* style: formatting code

* added scripts for centralized connections

* feat: centralized-connection javascore tests added

* feat: clam fees tests added

---------

Co-authored-by: red__larva <red.x.larva@proton.me>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* fix: centralized-connection test fixed

* fix: proxy request structure with data hash added

* fix: getFee method

* fix: message encoding decoding fixed

* fix: logs removed

* feat: response message solidity

* fix: review changes

* feat: tests added

* fix merge issues

---------

Signed-off-by: gcranju <qa5505815@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>
Co-authored-by: DeepakBomjan <44976635+DeepakBomjan@users.noreply.github.com>
Co-authored-by: izyak <76203436+izyak@users.noreply.github.com>
Co-authored-by: red__larva <red.x.larva@proton.me>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ibrizsabin ibrizsabin ibrizsabin left review comments

@MuhammedIrfan MuhammedIrfan MuhammedIrfan approved these changes

Assignees

@shreyasbhat0 shreyasbhat0

Labels
Feature HugoByte
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@shreyasbhat0 @codecov-commenter @MuhammedIrfan @ibrizsabin