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

Add first version of plugin-trait. #27

Merged
merged 2 commits into from
May 26, 2021
Merged

Add first version of plugin-trait. #27

merged 2 commits into from
May 26, 2021

Conversation

LNow
Copy link
Contributor

@LNow LNow commented Apr 28, 2021

Initial version of plugin-trait that we can use to implement Smart Contract Plugin System

close #26

@LNow LNow added the 👀 Clarity Something todo with Clarity label Apr 28, 2021
Zk2u
Zk2u previously approved these changes May 4, 2021
View reviewed changes
Copy link

@Zk2u Zk2u left a comment

Choose a reason for hiding this comment

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

LGTM for a first draft

cryptopanter
cryptopanter requested changes May 5, 2021
View reviewed changes
Copy link
Member

@cryptopanter cryptopanter left a comment

Choose a reason for hiding this comment

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

Changing trait to standard and allocating Standard identifiers (SyvR-v1)

@Zk2u
Copy link

Zk2u commented May 5, 2021 •
edited by cryptopanter
Loading

Changing trait to standard and allocating Standard identifiers (SyvR-v1)

Is this not a dados only thing? SyvRs are for organisation code standards, etc?
What i see it's a general structure for plugins... Is it sth specially for DAO?

@Zk2u Zk2u added ⚠️ High Priority High priority issue 💡 Feature A new feature or request labels May 17, 2021
@cryptopanter cryptopanter self-requested a review May 24, 2021 13:42
@Zk2u Zk2u merged commit 325aa1f into main May 26, 2021
@Zk2u Zk2u deleted the feat/plugin-trait branch May 26, 2021 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JMKJR JMKJR JMKJR approved these changes

@Zk2u Zk2u Zk2u approved these changes

@cryptopanter cryptopanter cryptopanter approved these changes

@friedger friedger Awaiting requested review from friedger

Assignees

@JMKJR JMKJR

Labels
👀 Clarity Something todo with Clarity 💡 Feature A new feature or request ⚠️ High Priority High priority issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create plugin standard
4 participants
@LNow @Zk2u @JMKJR @cryptopanter