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

feat(gui): remove assumptions about admins methods #76

Closed
wants to merge 1 commit into from

Conversation

chadoh
Copy link
Contributor

@chadoh chadoh commented Apr 21, 2022

This removes entanglement between what will soon become a
general-purpose frontend and the admins behavior of the TenK contract.
Instead, it only assumes that a specific comment format is followed in
the contract. In the future these comments will be generated in the wit
if an admins macro is present for the given contract method.

Now Change Methods in the UI will be longer or shorter, depending on
which methods the signed-in user is able to call.

demo

@chadoh chadoh requested a review from willemneal April 21, 2022 13:28
@chadoh chadoh force-pushed the feat/no-contract-assumptions branch 3 times, most recently from a3767af to fa897f5 Compare April 21, 2022 14:32
@github-actions
Copy link

github-actions bot commented Apr 21, 2022

PR Preview Action v1.1.1
🚀 Deployed preview to https://TENK-DAO.github.io/tenk/pr-preview/pr-76/
on branch gh-pages at 2022-05-04 15:13 UTC

@chadoh chadoh force-pushed the feat/no-contract-assumptions branch from fa897f5 to 0256a07 Compare May 4, 2022 15:02
This removes entanglement between what will soon become a
general-purpose frontend and the admins behavior of the TenK contract.
Instead, it only assumes that a specific comment format is followed in
the contract. In the future these comments will be generated in the wit
if an admins macro is present for the given contract method.

Now `Change Methods` in the UI will be longer or shorter, depending on
which methods the signed-in user is able to call.
@chadoh chadoh force-pushed the feat/no-contract-assumptions branch from 0256a07 to 90845c6 Compare May 4, 2022 15:10
@willemneal willemneal closed this Jun 14, 2022
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.

None yet

2 participants