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

Improve how we communicate what contributes to the storage quota to users #2081

Closed
mbommerez opened this issue Oct 28, 2022 · 2 comments · Fixed by #2101 or #2136
Closed

Improve how we communicate what contributes to the storage quota to users #2081

mbommerez opened this issue Oct 28, 2022 · 2 comments · Fixed by #2101 or #2136
Labels
topic/pot Issues handled by PT.

Comments

@mbommerez
Copy link

mbommerez commented Oct 28, 2022

Challenge

Once we merge #2069 we will start counting failed/incompleted uploads toward the calculated storage quota for a user. These failed uploads are shown in the table.
This might not be obvious to users, and, I wonder if the paid plans makes this a bit more sensitive.

Suggested solution

Let's add a tooltip to explain to the user what accrues to the total, making sure they are aware that our current storage calculation also includes incompleted uploads.

Suggested copy

This is the total of all your uploads, including the ones that never completed.
You can remove uploads via the table below, through your preferred client library, the CLI or the HTTP APIs.

We tried to keep it as compact as possible, given this goes into a tooltip.

How it looks in place

Here's a screenshot of how this could look in the dashboard

Image

Questions

We are wondering if we should suggest to users to reach out if they need help with bulk deletion, given there's no trivial way to do it with the APIs right now.
It could be something along the lines of:

Please get in touch with support@web3.storage if you need bulk deleting files.

At the same time, not sure if that's required and might create more work for you @dchoi27

What do you think?

Follow up on #2033

@mbommerez mbommerez added kind/bug A bug in existing code (including security flaws) need/triage Needs initial labeling and prioritization labels Oct 28, 2022
@mbommerez mbommerez changed the title Double check on failed uploads not counting towards calculated storage Improve user calculated storage - failed uploads taken into account Oct 28, 2022
@flea89 flea89 changed the title Improve user calculated storage - failed uploads taken into account Improve how we communicate what contributes to the storage quota to users Oct 28, 2022
@mbommerez
Copy link
Author

Let's add a warning / tooltip for user to be aware that our current storage calculation also includes failed uploads. And add 'Please contact support if you need help with removing those'.

@dchoi27
Copy link
Contributor

dchoi27 commented Nov 9, 2022

Hey - I think the thought here is fine, though we're trying to move away from telling the user when their upload is "completed" since in the future they could be uploading partial graphs on purpose. I'd make the copy say something more like This is the total of all your uploads, including ones marked "Failed" (you can read more about what the status means [here](https://web3.storage/docs/how-tos/get-status/). You can remove uploads via the table below.

Definitely don't want to tell them to reach out if they need to bulk delete files though (let's actually remove this from the issue).

@flea89 flea89 removed their assignment Nov 11, 2022
flea89 added a commit that referenced this issue Nov 30, 2022
Resolves #2081

Co-authored-by: Adam Alton <270255+adamalton@users.noreply.github.com>
Co-authored-by: David Choi <dchoi27@gmail.com>
adamalton pushed a commit that referenced this issue Dec 13, 2022
🤖 I have created a release *beep* *boop*
---


##
[2.33.0](website-v2.32.0...website-v2.33.0)
(2022-12-01)


### Features

* add tooltip to dashboard storage totals
([#2101](#2101))
([2056ec6](2056ec6)),
closes [#2081](#2081)
* list and ability to delete psa request created though deleted tokens
([#2054](#2054))
([e150d1f](e150d1f))
* simple analytics
([#2110](#2110))
([4d5a1a0](4d5a1a0))


### Bug Fixes

* styles missing from custom storage form
([#2135](#2135))
([04b66bc](04b66bc))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic/pot Issues handled by PT.
Projects
None yet
3 participants