feat: move reupload command to root level#774
Conversation
|
| }) | ||
| } | ||
| } catch (error) { | ||
| this.console.printBeeError(error, { notFoundMessage: 'No locally pinned content found with that address.' }) |
There was a problem hiding this comment.
This error message no longer makes sense as a root level command.
Something like "No content found with that address." would be better.
There was a problem hiding this comment.
Please revert this change. Our coverage tracking works in mysterious ways 😅 But it shouldn't be committed
| █ Available Commands: | ||
| reupload Reupload locally pinned content |
There was a problem hiding this comment.
The copy also needs a change here since we are no longer in the context of pinning. Suggested: "Reupload and restamp content on the network"
Cafe137
left a comment
There was a problem hiding this comment.
Thanks for adding the test, the functionality looks good.
Most of the comments are about removing the unrelated files that were committed.
The rest are about changing copies that still refer to pinning, which is not a requirement anymore for this command. Please also change the description field of the command to "Reupload and restamp content on the network"
No description provided.