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.

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

User input creator tx #942

Merged
merged 5 commits into from
Mar 9, 2023
Merged

User input creator tx #942

merged 5 commits into from
Mar 9, 2023

Conversation

kuzdogan
Copy link
Member

@kuzdogan kuzdogan commented Mar 2, 2023

Changes enable passing the creatorTxHash variable to server. Normally this is scraped from an explorer or fetched with an API but sometimes this fails. This allows users to provide it themselves.

  • Adds the variable to both session and legacy verify endpoints
  • Adds tests for both
  • Additionally refactors how files are saved to remove the redundant code.

TODO:

  • Add the field to the docs once the PR passes.

View in Huly HI-734

kuzdogan added 5 commits March 1, 2023 16:13
To keep the exact content of the JSON file consistent, it is better to
stringify the JSON without adding tabs and spaces, just as
we do with the metadata.json
Different store methods for metadata, library-map etc. was redundant.

Essentially there are:
- sources
- JSON files
- Txt files

The number of functions can be reduced without losing readability easily.
Lets passing the creatorTxHash variable in the API.

Saves the creatorTxHash inside "creator-tx-hash.txt" file in the
contract folder.

Adds the server tests for verifying a withImmutable contract.
@kuzdogan kuzdogan requested a review from marcocastignoli March 2, 2023 10:14
@kuzdogan kuzdogan merged commit 92b65d3 into staging Mar 9, 2023
@kuzdogan kuzdogan deleted the pass-creatorTx branch March 29, 2023 07:29
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.

2 participants