Skip to content
This repository has been archived by the owner on Feb 3, 2023. It is now read-only.

Init replaces genesis #1508

Merged
merged 38 commits into from Jul 24, 2019
Merged

Init replaces genesis #1508

merged 38 commits into from Jul 24, 2019

Conversation

willemolding
Copy link
Collaborator

PR summary

Since we are no longer going to have a genesis function but rather a validate capability grant callback I think it makes sense to merge this now.

Init replaces genesis in define_zome! and proc macro HDK. It takes some init params which is a JsonString entered in the conductor config.

It is also now possible to call hdk functions in init since it is no longer performing validation. init is more like a regular zome function that is called on startup. The membrane enforcement is now done by the validate_agent and soon the validate_capability callbacks.

changelog

Please check one of the following, relating to the CHANGELOG-UNRELEASED.md

  • this is a code change that effects some consumer (e.g. zome developers) of holochain core so it is added to the CHANGELOG-UNRELEASED.md (linked above), with the format - summary of change [PR#1234](https://github.com/holochain/holochain-rust/pull/1234)
  • this is not a code change, or doesn't effect anyone outside holochain core development

Copy link
Collaborator

@lucksus lucksus left a comment

Choose a reason for hiding this comment

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

Looks great, except for the wrong link in the changelog.

CHANGELOG-UNRELEASED.md Outdated Show resolved Hide resolved
lucksus and others added 2 commits July 23, 2019 13:06
Copy link
Contributor

@StaticallyTypedAnxiety StaticallyTypedAnxiety left a comment

Choose a reason for hiding this comment

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

This looks good to me!

@willemolding willemolding merged commit f6d4e71 into develop Jul 24, 2019
@zippy zippy deleted the init-replaces-genesis branch October 4, 2019 18:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants