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.

Sign up for GitHub

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

Jump to bottom

docs: improved "Cannot process IPFS request" page #911

Merged
merged 1 commit into from
Jul 20, 2020
Merged

docs: improved "Cannot process IPFS request" page #911

merged 1 commit into from
Jul 20, 2020

Conversation

jessicaschilling
Copy link
Contributor

This PR search/replaces globally: bafkreiewrj2pugsghd3flw2lk2fhvtmz6wipecnxep5qc5m3lfpf2mvjk4 -> QmVGC4uCBDVEhCzsaJmvR5nVDgChM97kcYNehVm7L9jxtc

https://gateway.ipfs.io/ipfs/QmVGC4uCBDVEhCzsaJmvR5nVDgChM97kcYNehVm7L9jxtc is an improved "Cannot process IPFS request" page with better information hierarchy and clearer guidance for next steps.

Local screenshots with fake URIs bashed in for the sake of illustration:

image
image

Closes #815

@jessicaschilling jessicaschilling requested a review from lidel July 16, 2020 18:27
@jessicaschilling
Copy link
Contributor Author

@lidel -- this takes care of @andrew's issue in #815. Please feel free to merge if you're OK with it!

@jessicaschilling jessicaschilling mentioned this pull request Jul 16, 2020
Closed
2 tasks
lidel
lidel approved these changes Jul 20, 2020
View reviewed changes
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.

👌 thank you!

@lidel lidel changed the title Clarify guidance on "Cannot process IPFS request" page docs: improved "Cannot process IPFS request" page Jul 20, 2020
@lidel lidel merged commit 6c05feb into master Jul 20, 2020
@lidel lidel deleted the bug/cannot-process-request-error-page branch July 20, 2020 15:51
@lidel lidel mentioned this pull request Nov 30, 2020
Closed
This was referenced Jan 5, 2021
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lidel lidel lidel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ipfs://CIDv0 does not work in Firefox 70
2 participants
@jessicaschilling @lidel