Skip to content

fix: CLI tutor commands missing some parameters#2470

Merged
lidel merged 1 commit into
ipfs:mainfrom
imrehg:cli-tutor-fixes
Feb 4, 2026
Merged

fix: CLI tutor commands missing some parameters#2470
lidel merged 1 commit into
ipfs:mainfrom
imrehg:cli-tutor-fixes

Conversation

@imrehg

@imrehg imrehg commented Feb 4, 2026

Copy link
Copy Markdown
Contributor

The "Download CAR" command and the "Publish with IPNS" commands were missing some of the values that they needed to be complete. Adding those in.

The results look as the below screenshots for the two affected CLI tutor commands:

Download CAR:

Screenshot_2026-02-04_23-09-10

Pin to IPNS:

Screenshot_2026-02-04_23-08-56

See the previous views in the linked issue below.

There's also a "@todo" item in this modified file:

// @TODO: ensure path is set for all actions

If there are other commands to cover, maybe we can add them here too, and remove the TODO. Will check that in more detail, while waiting for feedback on the changes.

Closes #2469

The "Download CAR" command and the "Publish with IPNS" commands
were missing some of the values that they needed to be complete.
Adding those in.
@imrehg imrehg requested a review from a team as a code owner February 4, 2026 15:12

@lidel lidel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thank you lgtm, feel free to improve other ones (if still applicable) in separate PRs

@lidel lidel merged commit ed8ad6a into ipfs:main Feb 4, 2026
11 checks passed
ipfs-gui-bot pushed a commit that referenced this pull request Feb 5, 2026
## [4.11.0](v4.10.0...v4.11.0) (2026-02-05)

 CID `bafybeidfgbcqy435sdbhhejifdxq4o64tlsezajc272zpyxcsmz47uyc64`

 ---

### Features

* Add search/filter functionality to Files UI ([#2451](#2451)) ([c866be6](c866be6)), closes [#2447](#2447)
* DHT Provide Sweep Diagnostic Screen ([#2463](#2463)) ([fb22ea6](fb22ea6))
* **files:** resolve paths before inspect and support protocol URL ([#2465](#2465)) ([74a44d8](74a44d8))
* **files:** support additional image file extensions ([#2347](#2347)) ([371341a](371341a))

### Bug Fixes

* **ci:** test stability improvements ([#2466](#2466)) ([d11475a](d11475a))
* CLI tutor commands missing some parameters ([#2470](#2470)) ([ed8ad6a](ed8ad6a))
* **diagnostics:** handle Go zero time in DHT provide screen ([dc51cd4](dc51cd4))
* **files:** not found page ([#2455](#2455)) ([18b9b0d](18b9b0d))
* show proper error state in import notifications ([#2452](#2452)) ([391470e](391470e)), closes [#2448](#2448)

### Trivial Changes

* **ci:** skip publishPreview for dependabot PRs ([17f675e](17f675e))
* pull new translations ([#2467](#2467)) ([cc569f4](cc569f4))
* pull transifex translations ([#2464](#2464)) ([8d7a17f](8d7a17f))
@ipfs-gui-bot

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 4.11.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@imrehg imrehg deleted the cli-tutor-fixes branch February 5, 2026 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: CLI tutor mode doesn't seem to fill in object ID correctly for Export CAR

3 participants