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

Clean old pins (ON HOLD) #262

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

pablomendezroyo
Copy link
Contributor

@pablomendezroyo pablomendezroyo commented Jul 15, 2022

Clean pins older than 30 days. There must never been unpinned release hashes

@pablomendezroyo pablomendezroyo marked this pull request as draft July 15, 2022 12:59
@github-actions
Copy link

github-actions bot commented Jul 15, 2022

DAppNode bot has built and pinned the release to an IPFS node, for commit: 0105797

This is a development version and should only be installed for testing purposes, install link

/ipfs/QmbKZ5snFeHufWGcgJT3LcFYWjtNtpZ677iNRfcETMseSQ

(by dappnodebot/build-action)

@pablomendezroyo pablomendezroyo marked this pull request as ready for review July 18, 2022 14:32
/**
* Returns all the IPFS hashes from a given package dnpName
*/
async getIpfsHashesFromDnpName(dnpName: string): Promise<string[]> {
Copy link
Member

Choose a reason for hiding this comment

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

Here you need to add hashes which were uploaded using old method. For example, this is the hash saved onchain of one of the first releases of DAppManager: https://gateway.ipfs.dappnode.io/ipfs/QmdjatAF1DeifJyZNLtefWsXDPZgFSGFU3e19QGsjYkGih
As you can see, that is just the manifest. So you need to parse that manifest and get image and avatar hash from there.

@pablomendezroyo pablomendezroyo changed the title Clean old pins Clean old pins (ON HOLD) May 9, 2023
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.

2 participants