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

feat: Use Blob protocol #118

Closed
wants to merge 23 commits into from
Closed

feat: Use Blob protocol #118

wants to merge 23 commits into from

Conversation

Peeja
Copy link
Member

@Peeja Peeja commented Aug 30, 2024

Waiting for @w3ui/[email protected]

  • Uses the updated React toolkit with the updates client, and therefore the Blob protocol.
  • Updates everything else required to the latest version to match. (Note that this is a smell and should be addressed, separately.)

As of PR draft, it depends on @w3ui/[email protected], which doesn't exist yet. Once that's published, this should be re-pnpm installed to get a correct lockfile, and then amended.

Closes storacha/project-tracking#123

Alan Shaw and others added 9 commits July 31, 2024 19:19
This was mainly a sweep of uses of `h-screen`, as it was breaking on
viewports shorter than the page contents, which was particularly nasty
on the Pricing page, which is long. I've also:

* Tweaked how the rooster shows up on small-width screens
* Made the "Terms of Service" text on the login screen black rather than
white for readability
* Made the hamburger menu icon red rather than write for readability
* Changed "w3up" to "Storacha"

Unfortunately, we don't have Storybook and Chromatic (or similar) to see
all the diffs, but I've screenshotted what I could. Note that these are
all full-page screenshots; the fact that the befores are cut off at the
top is exactly the problem I was solving here. 🙂

Plenty of this is subjective; thoughts, feedback, and corrections are
more than welcome.

### Before


![screencapture-localhost-3000-2024-08-02-16_00_56](https://github.com/user-attachments/assets/8f29fe4a-d078-4271-a3de-75ba30bbf67e)

![screencapture-localhost-3000-2024-08-02-16_02_16](https://github.com/user-attachments/assets/3b9f7b06-0d4d-401e-a90e-144d724f614a)

![screencapture-localhost-3000-2024-08-02-16_15_53](https://github.com/user-attachments/assets/78f31bfa-301c-440e-bdcf-e25962dd766a)

![screencapture-localhost-3000-2024-08-02-16_16_38](https://github.com/user-attachments/assets/a3c09240-a1d4-4029-81c0-6df4a3943a27)

![screencapture-localhost-3000-2024-08-02-16_27_53](https://github.com/user-attachments/assets/1fc08c56-0a58-46aa-939d-7b9aab5eafad)

### After


![screencapture-localhost-3000-2024-08-02-16_01_30](https://github.com/user-attachments/assets/39ec5a46-2dd2-407e-afc9-746e7288447b)

![screencapture-localhost-3000-2024-08-02-16_07_23](https://github.com/user-attachments/assets/78435c8c-d862-4051-ad2a-f51f3bb87a16)

![screencapture-localhost-3000-2024-08-02-16_16_06](https://github.com/user-attachments/assets/365539c3-5966-4885-bd63-dd51f4cfa3f7)

![screencapture-localhost-3000-2024-08-02-16_16_56](https://github.com/user-attachments/assets/dfdb7587-f06f-48cc-9dee-dcc1fff77a99)

![screencapture-localhost-3000-2024-08-02-16_28_11](https://github.com/user-attachments/assets/850af45e-b073-4a09-a420-79245a354c45)
@Peeja Peeja marked this pull request as draft August 30, 2024 16:51
Copy link
Member

@alanshaw alanshaw left a comment

Choose a reason for hiding this comment

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

There's some UI work to be done here.

When viewing a shard i.e. /space/:spaceDID/root/:rootCID/shard/:carCID we use store/get to obtain the shard size. We need this to be switched to space/blob/get (with fallback to store/get on 404).

https://github.com/storacha-network/console/blob/5f135df3ff80f62827c3fccd4ded2667dd144b49/src/app/space/%5Bdid%5D/root/%5Bcid%5D/shard/%5Bshard%5D/page.tsx#L39-L50

fforbeck and others added 7 commits September 17, 2024 14:05
# Issue
If the `NEXT_PUBLIC_W3UP_PROVIDER` environment variable is not set, the
app will use `did:web:web3.storage` as the default storage provider.
That works well if we are pointing to Prod environment
(https://up.web3.storage), however, if we use the Staging environment
(https://staging.up.web3.storage) it will fail to provision the space
because the default provider is invalid for Staging.

# Fix 
- Verify if the Space Provisioning result contains an error. If so,
throw the error to prevent the user from continuing the flow and
attempting to upload files.
- Updated Docs and sample Env Vars.

---------

Co-authored-by: Alan Shaw <[email protected]>
So we're all on the same version (and package manager).
Just a tiny copy change.
I've updated the workflow to use the `old-main` branch to deploy the
`https://console.web3.network` app.
I've fixed the `deploy-storacha.yml` workflow to build previews and
changelogs, run tests, and create releases using the same conditionals
we had while deploying to `https://console.web3.storage`.
# Goals

use up.storacha.network for service URL

# Implementation

update deploy scripts in github.
🤖 I have created a release *beep* *boop*
---


##
[1.10.0](w3console-v1.9.1...w3console-v1.10.0)
(2024-09-20)


### Features

* change email
([7c7beb7](7c7beb7))
* **copy:** Add missing comma
([#115](#115))
([97f1ac3](97f1ac3))
* Improved layout styling, and other tales
([#114](#114))
([d0a2208](d0a2208))
* more styles
([d4e8efa](d4e8efa))
* styles for upload page
([6c54798](6c54798))
* trigger deploy
([a8f1089](a8f1089))
* wording
([ffe9b1b](ffe9b1b))


### Bug Fixes

* apostrophe
([681b9b8](681b9b8))
* check ref
([f406ac9](f406ac9))
* deploy-storacha.yml needs to use the new main branch
([#122](#122))
([65644c8](65644c8))
* deploy.yml needs to use the old-main branch
([#120](#120))
([713af05](713af05))
* quotes
([bcf209b](bcf209b))
* **space-creator:** check provision result before proceeding
([#119](#119))
([5ea5940](5ea5940))
* use correct branch name
([f81e090](f81e090))
* use main branch
([698974b](698974b))


### Other Changes

* add storacha deploy action
([39345e6](39345e6))
* Set `packageManager`
([#117](#117))
([c9436fc](c9436fc))
* update deploy name
([f7c1340](f7c1340))

---
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>
@fforbeck
Copy link
Member

fforbeck commented Sep 23, 2024

@Peeja I've released the @w3ui/react and @w3ui/core last week. Version 2.4.0

fforbeck and others added 7 commits September 24, 2024 10:52
### Add Share via Email Feature

**Context**:  
This PR introduces the Share via Email feature, allowing users to share
spaces using either an email address or DID. Depending on the input, we
render different buttons and actions (e.g., sharing via email or
generating a downloadable UCAN). Once shared via email, the “Shared
With” panel lists all shared emails (did:mailto) along with their
capabilities, fetched from the delegations stored in the client.

**Short Demo**:

https://www.loom.com/share/b33ee2f22a5b44628e5d9fc3d599dbf7?sid=0b008211-fc28-4cd4-a716-e67f44067661

**Minor Notes**:
- The Share button dynamically changes based on whether the user
provides an email or DID.
- The delegation list is displayed with relevant capabilities in the
“Shared With” panel.
Fix for issues
- storacha/project-tracking#131
- #93

The issue happens when we try to delegate `*` to the DID of the
audience. I've updated the code to delegate the same capabilities we use
in the `client.shareSpace` method.
Fixes: #94

When you import an Space from the UCAN file it generates several
components as Space Preview:
![Screenshot from 2024-09-20
15-44-05](https://github.com/user-attachments/assets/df94198f-e635-4bf9-8d1c-7b92a8d3c3f8)

Ideally, it would render only one component for the Space that was
imported:
![Screenshot from 2024-09-23
11-02-22](https://github.com/user-attachments/assets/e0904f9a-cef3-4218-889f-0c2249169e33)

This PR fixes that component and includes the information icon to show
the capabilities delegated to that shared Space.
🤖 I have created a release *beep* *boop*
---


##
[1.11.0](w3console-v1.10.0...w3console-v1.11.0)
(2024-09-24)


### Features

* share spaces via email
([#123](#123))
([896079b](896079b))


### Bug Fixes

* import space preview
([#127](#127))
([1f2acf6](1f2acf6))
* share space via ucan file
([#126](#126))
([ba934a2](ba934a2))

---
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>
<img width="1356" alt="Screenshot 2024-09-25 at 18 34 57"
src="https://github.com/user-attachments/assets/eaefb7b7-9231-4823-a2f2-b31d87d94457">

---------

Co-authored-by: Alan Shaw <[email protected]>
🤖 I have created a release *beep* *boop*
---


##
[1.12.0](w3console-v1.11.0...w3console-v1.12.0)
(2024-10-02)


### Features

* add old web3 storage migrator
([#129](#129))
([ba45a5f](ba45a5f))

---
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>
Copy link
Contributor

Website preview 🔗✨

build log

Peeja added a commit that referenced this pull request Oct 11, 2024
Reopens #118 against `main`, and
should address @alanshaw's
[feedback](#118 (review))
(making the Shard page work again).
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.

The Console uses the blob protocol
4 participants