Skip to content

feat(FullAccessKeyFallback)!: remove the plugin#87

Merged
mooori merged 5 commits into
masterfrom
rm-fak
Mar 10, 2023
Merged

feat(FullAccessKeyFallback)!: remove the plugin#87
mooori merged 5 commits into
masterfrom
rm-fak

Conversation

@mooori
Copy link
Copy Markdown
Contributor

@mooori mooori commented Mar 8, 2023

Removes the FullAccessKeyFallback since there are some issues with its design: external-4, external-6, external-23, external-30.

The default implementation of FullAccessKeyFallback consists only of a few lines of code. If smart contract devs need this behavior, it makes more sense for them to implement their own solution tailored to their use-case.

Breaking changes

It removes a plugin: the FullAccessKeyFallback trait and its default implementation.

Additional changes

FullAccessKeyFallback was the last plugin that was included in examples/. As the corresponding examples were removed, some code became unused, which is removed in this PR. This allows deleting the examples/ directory.

The contracts in near-plugins-derive/tests/contracts serve as examples now (ref. #48).

@mooori mooori marked this pull request as ready for review March 10, 2023 10:31
@mooori mooori requested a review from birchmd March 10, 2023 10:31
@mooori mooori merged commit 8619c33 into master Mar 10, 2023
@mooori mooori deleted the rm-fak branch March 10, 2023 13:50
This was referenced Mar 27, 2026
This was referenced Apr 7, 2026
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.

3 participants