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

Feature Request: Add link to pinning service compliance github pages summary in ipfs-webui pinning popup #30

Open
1 of 2 tasks
SgtPooki opened this issue Apr 28, 2022 · 7 comments

Comments

@SgtPooki
Copy link
Member

SgtPooki commented Apr 28, 2022

In the ipfs UI files page, you can select to pin your ipfs files.
image

(When we show remote pinning services) We should link to the pinning-service-compliance results pages so that users can make a more educated decision.

Short term:

  • Simple link to github pages summary of all pinning services for this repo

Long term:

  • expose json results somehow so results can be consumed and the file pinning popup can display compliance results directly within the popup (and we can recommend fully compliant services over non-fully-compliant services)
@SgtPooki
Copy link
Member Author

@lidel IIRC, we've talked about this previously, but I don't believe we've documented anything about it at all.

@SgtPooki
Copy link
Member Author

SgtPooki commented Jun 8, 2022

PR is out for short-term fix: ipfs/ipfs-webui#1943

@SgtPooki SgtPooki pinned this issue Jun 14, 2022
@SgtPooki
Copy link
Member Author

the short term fix is done since ipfs/ipfs-webui#1943 was merged.

@SgtPooki SgtPooki modified the milestones: Initial release, v2 Jun 15, 2022
@lidel
Copy link
Member

lidel commented Jun 23, 2022

Been thinking about user flow here, and I am unsure what is the value in showing this in pinning popup – user's will be just annoyed, feels like GDPR cookie noise a bit 🙈 😅

What if.. we remove it from this popup and add it to the Pinning Services section in Settings, so it is shown not only when user adds a new one, but every time they look at that table?

@SgtPooki
Copy link
Member Author

I think it would make sense in both places.

@juliaxbow any thoughts? Lmk if you need more context

@juliaxbow
Copy link

Assuming there are real risks to inadvertently selecting a non-compliant pinning service, I'm +1 to including help text/linking out to compliance reports in the pop up. Users won't be required to take action (ie acknowledge or close out of a window) so it won't be as annoying as GDPR notices.

That said, I also realize that you can't add new pinning services in the pop-up and users should be encouraged to do compliance checks at the stage of adding a new service. So, an alternative to consider is to add help text along the lines of

Add pinning services and check compliance reports in settings

This solves for another kink in the current user flow, in which a new user goes to pin something and doesn't understand their pinning options beyond their local node unless they find their way to the settings page.

Pinning pop up with help text

Pinning pop up with help text - Annotated

@SgtPooki
Copy link
Member Author

I think that’s a good approach. I want to clarify that one thing i wanted to cover with this issue was displaying compliance percentages next to the provider.

If we show percentagss, and a provider becomes less compliant, a user who has already added that service will be aware. If we only ever show/link to compliance results when adding a provider then providers can change at any time with little disincentive.

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

No branches or pull requests

3 participants