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

Start Project: beta ag-solo dapp architecture notice #837

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

dckc
Copy link
Member

@dckc dckc commented Aug 4, 2023

Let devs know that this ag-solo / mailbox style is not how mainnet dapps work.

TODO:

  • link to permissioned deployment?
  • link to smart-wallet/RPC architecture

editorial tweaks moved to separate PRs

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 4, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: f1aac5c
Status: ✅  Deploy successful!
Preview URL: https://5df61ca7.documentation-7tp.pages.dev
Branch Preview URL: https://dc-permissioned-deployment.documentation-7tp.pages.dev

View logs

@dckc dckc force-pushed the dc-permissioned-deployment branch from 42e47be to fd84981 Compare August 4, 2023 05:33
@dckc dckc marked this pull request as draft August 4, 2023 15:08
Let devs know that this ag-solo / mailbox style is not how mainnet
dapps work.
@dckc dckc marked this pull request as ready for review November 14, 2023 21:58
@dckc dckc enabled auto-merge (squash) November 14, 2023 22:06
Copy link
Contributor

@kbennett2000 kbennett2000 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 to me!

@dckc dckc merged commit fcb94ac into main Nov 14, 2023
4 checks passed
@dckc dckc deleted the dc-permissioned-deployment branch November 14, 2023 22:20
@@ -3,6 +3,16 @@
After you've [installed the Agoric SDK](./README.md) (_recall: use `agoric --version` to confirm._), you're ready for your first _Agoric Dapp_ (decentralized application).


::: tip Beta Dapp Architecture
Copy link
Member

Choose a reason for hiding this comment

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

I am not familiar with this repo, is there an ::: alert or ::: infowe can use that's maybe more prominent? Definitely noticeable, but wondering if we can make it stand out a little more.

@dckc dckc mentioned this pull request Jan 25, 2024
This was referenced Mar 18, 2024
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