Skip to content
This repository has been archived by the owner on Dec 20, 2024. It is now read-only.

Update AppData Fetching #43

Open
bh2smith opened this issue Jun 8, 2023 · 0 comments
Open

Update AppData Fetching #43

bh2smith opened this issue Jun 8, 2023 · 0 comments

Comments

@bh2smith
Copy link
Contributor

bh2smith commented Jun 8, 2023

The backend is going to start encoding app data differently and we will need to change the inverse mapping

appHash --> CID

@vkgnosis shared the rust code that already does this here

and it appears we will just have to change the prefix here.

However we will need to continue to support the old style (forever) because old app hashes will always exist. Unless they plan to migrate all the old files to the new schema.

In terms of support there will be a block number for which the new content starts but is mixed (deployed in staging) with old and another block where we only support new app hashes. The code will need to know these blocks say (left, right) so that we use old on blocks below left, both on block between left and right and new on blocks after right. Might actually be easier to just check both always (with priority on the new schema).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant