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

remove defunct pinning services and do a bit of gardening #1902

Merged
merged 9 commits into from
Jul 24, 2024
Merged

Conversation

2color
Copy link
Member

@2color 2color commented Jul 23, 2024

What's in this PR

  • fixes Update concepts/persistence/#pinning-services list #1899
  • Remove mentions of spheron which no longer offers IPFS
  • Remove unhelpful pages with outdated links
  • Remove welcome builders page which isn't linked to from anywhere (has almost no visits in analytics and 100% bounce rate)
  • correct Elastic IPFS name
  • Fix spell checker CI which broke when vale configuration

@2color 2color requested a review from lidel July 23, 2024 14:52
@2color 2color changed the title chore: remove defunct pinning services remove defunct pinning services and do a bit of gardening Jul 23, 2024
Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

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

lgtm, thank you for cleaning these up

Copy link
Member

Choose a reason for hiding this comment

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

💭 I liked the visual of this page.
We may want reuse the ipfs-builders-rocket.svg image somewhere else :)

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, this feels more like inspirational content for ipfs.tech website, rather than technical docs.

@lidel lidel merged commit 01c955f into main Jul 24, 2024
6 checks passed
@lidel lidel deleted the gardening-1 branch July 24, 2024 14:15
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.

Update concepts/persistence/#pinning-services list
2 participants