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

Split out CSS-specific instructions from self-hosting overview #690

Closed

Conversation

michielbdejong
Copy link
Contributor

No description provided.

@michielbdejong michielbdejong force-pushed the fix-689-self-hosting-instructions branch from 214341a to 89ba76f Compare April 19, 2022 08:58
@michielbdejong
Copy link
Contributor Author

This fixes #689.

@timbl
Copy link
Contributor

timbl commented May 4, 2022

The move of infomration into more logical set of pages does not have to wait for test stuff to be decided, surely?

@timbl timbl reopened this May 4, 2022
@csarven
Copy link
Member

csarven commented May 4, 2022

With the exception of updating as per #690 (comment) , this PR is fine as far reorganising existing information goes. solidproject.org happens to include the instructions to help potential users but if that information is subject to going stale or better to link out to an authoritative document that's published by the CSS team somewhere and updated more frequently, then that can work too.

@RubenVerborgh RubenVerborgh dismissed their stale review May 9, 2022 19:29

dismiss

@michielbdejong michielbdejong requested a review from csarven May 10, 2022 07:37
Copy link
Member

@csarven csarven left a comment

Choose a reason for hiding this comment

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

I think this PR is no longer applicable as the site seems to have moved on with other changes (from other PRs): https://solidproject.org/users/get-a-pod#self-host-your-pod

@michielbdejong , can we close this?

Sorry that this PR was closed, reopened, and then gone stale. Solid Team should better process open PRs...

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.

4 participants