Skip to content

Conversation

dckc
Copy link
Member

@dckc dckc commented Mar 21, 2024

closes #947

please excuse incremental work toward...

turning whitespace off in the diff should help.

subsumes:

@dckc dckc changed the base branch from main to dc-vstorage-ref March 21, 2024 13:45
@dckc dckc force-pushed the dc-vstorage-guide branch from f776e07 to e119387 Compare March 21, 2024 13:48
Copy link

cloudflare-workers-and-pages bot commented Mar 21, 2024

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8c3dfaf
Status:⚡️  Build in progress...

View logs

@dckc dckc changed the base branch from dc-vstorage-ref to main March 21, 2024 14:08
@dckc dckc mentioned this pull request Mar 21, 2024
@dckc
Copy link
Member Author

dckc commented Mar 21, 2024

vstorage reference research notes

I'm about to close (and maybe delete) Agoric/agoric-sdk#9115 , but some of it didn't make it into the PR, so let's keep it here...

@gibson042 wrote:

Yeah, I think you've pretty much got it, although it's worth noting that everything except highPrioritySenders (and maybe that as well) predates published, which I introduced to avoid collisions at the top level. Details that I know about:

I'm not aware of docs on any of them, nor even which are actively in use.

@dckc dckc changed the title docs: publishing to vstorage using the chainStorage API docs: vstorage reference; chainStorage publishing guide Mar 21, 2024
@dckc dckc requested a review from Chris-Hibbert March 21, 2024 19:44
@Chris-Hibbert
Copy link
Contributor

The preview seems broken. I find it easier to review with that open. Would you repair?

@dckc
Copy link
Member Author

dckc commented Mar 21, 2024

The preview seems broken.

oops! broken link.
it's rebuilding now.

**Warning: this includes the right to over-write previously allocated storage nodes.**

- **chainTimerService**: for getting the current timer and setting timer wake-ups; for example, at the conclusion of a governance vote.
- **chainTimerService**: for getting the current [timer](../../reference/repl/timerServices) and setting timer wake-ups; for example, at the conclusion of a governance vote.
Copy link
Contributor

Choose a reason for hiding this comment

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

Aargh! That's way out-of-date. I added a link to the internal doc to #948.

}
```

The `published.wallet.${address}` key has wallet last update. For example:
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this accurate? Something was missing.

Suggested change
The `published.wallet.${address}` key has wallet last update. For example:
The `published.wallet.${address}` key has the wallet's last update. For example:

as long as either it gets the assets described in the want section of its proposal, or it retains
all of the assets specified in the give section of the proposal. This constraint applies to each
seat across the entire atomicRearrangement, not to the individual **TransferParts**.
Stagings are a reallocation mechanism that has been

Copy link
Contributor

Choose a reason for hiding this comment

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

This is still broken. Compare:
stagings-wrong

== with ======

Stagings-orig

@dckc dckc requested a review from Chris-Hibbert March 22, 2024 19:20
@dckc dckc force-pushed the dc-vstorage-guide branch from 577237c to 8c3dfaf Compare March 22, 2024 22:39
@dckc dckc merged commit 44e1071 into main Mar 22, 2024
@dckc dckc deleted the dc-vstorage-guide branch March 22, 2024 22:40
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.

Vstorage journey - 4daydapp
2 participants