fix: naming consistency in messaging (messageHash -> message)#641
Merged
Conversation
jeanmon
pushed a commit
that referenced
this pull request
May 23, 2023
* fix: contentHash -> content * fix: update content in nr * fix: update incorrect message hash naming --------- Co-authored-by: Maddiaa0 <addaboy@gmail.com>
suyash67
pushed a commit
that referenced
this pull request
May 25, 2023
* prelim / very-broken commit where Jean and I started implementing the initial private kernel and moving logic between it and the inner kernel * wip - fix some compilation issues, remove unused imports, remove update_end_values leftover * wip - some minor changes and comments after pairing with David * wip: additional compilation fixes * Refactor helper functions in private kernel tests * wip - Fix linking issue and comment out some tests * wip - repair native and circuit tests * wip - include using double quotes and remove unused code * wip - fixing circuit_create_proof_cbinds test and adapting private_kernel_sim function to the inner/init split * wip - fix clang tidy issue * wip - post master merger fixes and remove test on new nullifiers * wip - fix clang tidy issue * wip - fix clang tidy issue * wip - fix typescript tests and data structure split * wip - format typescript * wip - replace CONSTRUCTOR_ARGS by FUNCTION_ARGS * wip - add nullifier and l1-l2-message tree roots in initial private kernel circuit * wip - post merge fix * Migrate common.hpp functions into non-templated ones * refactor(sol): use Hash.sha256ToField library where required (#637) * refactor(sol): use Hash.sha256ToField library where required * fmt :) * feat: expose bytes32 hash directly * fmt: https://i.pinimg.com/originals/81/23/a1/8123a132c007eab782d6ca9bed517eb3.jpg --------- Co-authored-by: Maddiaa0 <addaboy@gmail.com> * fix: Add checks for arg-type (#638) * fix: Add checks for arg-type * fix: support arg of 'number' type * fix: correct wrapped fr in acir sim * no need to force encode in some e2e tests! --------- Co-authored-by: Maddiaa0 <addaboy@gmail.com> Co-authored-by: Rahul Kothari <rahul.kothari.201@gmail.com> * fix: naming consistency in messaging (messageHash -> message) (#641) * fix: contentHash -> content * fix: update content in nr * fix: update incorrect message hash naming --------- Co-authored-by: Maddiaa0 <addaboy@gmail.com> * chore: remove unknown casts in archiver test (#648) Removes intermediate casts to unknown from archiver test, which hide type errors in the args of the mocked event. * Consolidation code for contract_logic * e2e fixing: Initialise public and private call stack and comment out some checks * Renaming file native_private_kernel_initial.XPP into native_private_kernel_init.XPP * Rename native_private_kernel_circuit into native_private_kernel_circuit_inner * Simplify if/else clause on deployment contract boolean in contract logic * abis test snapshot - restore from master for vk hash * Remove old TODO comments * Fork FUNCTION_ARGS into FUNCTION/CONSTRUCTOR_ARGS generator indices * Move code copying public/private_call_stack in PKC init * Clang tidy fix * Add clarifications into a comment * Added comments * remove empty lines --------- Co-authored-by: jeanmon <jean@aztecprotocol.com> Co-authored-by: Maddiaa <47148561+Maddiaa0@users.noreply.github.com> Co-authored-by: Maddiaa0 <addaboy@gmail.com> Co-authored-by: Lasse Herskind <16536249+LHerskind@users.noreply.github.com> Co-authored-by: Rahul Kothari <rahul.kothari.201@gmail.com> Co-authored-by: Santiago Palladino <santiago@aztecprotocol.com>
This was referenced Jul 25, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
closes: #632
Very small pr correcting use of contentHash in L1ToL2 data types to make it consistent with uses elsewhere. ( referring to it as a hash is incorrect as in some instances, where the value fits within a field, it will not need to be hashed ).
Checklist: