Skip to content

Conversation

youhaveme9
Copy link
Member

Checklist

  • I have read the OpenWISP Contributing Guidelines.
  • I have manually tested the changes proposed in this pull request.
  • I have written new test cases for new code and/or updated existing tests for changes to existing code.
  • I have updated the documentation.

Reference to Existing Issue

Closes #346

Right now, the update of the upgrade information is not asynchronous yet,
so you will have to reload the page periodically to find new information.
This will be addressed in a future release.
The firmware upgrade process now features a realtime progress bar that
Copy link
Member

Choose a reason for hiding this comment

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

I would try to communicate this with less words and connect it to the previous sentence, eg:

then a new tab "<NAME OF TAB HERE>" will appear in the device page which allows us to
observe in real time what's going on during the upgrade without requiring page reloads.

<SCREENSHOT OR GIF FOCUSED ON THIS ASPECT HERE>

Adding another dedicated screenshot should help to give more visibility to this great work you've done.

Right now, the update of the upgrade information is not asynchronous yet,
so you will have to reload the page periodically to find new information.
This will be addressed in a future release.
The mass upgrade operation now features a realtime progress bar that
Copy link
Member

Choose a reason for hiding this comment

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

I would try to communicate this with less words and connect it to the previous sentence, eg:

you will be redirected to a page in which you can monitor the progress of the upgrade operations
in real time.

<SCREENSHOT OR GIF FOCUSED ON THIS ASPECT HERE>

Adding another dedicated screenshot should help to give more visibility to this great work you've done.

Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

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

Here we need to change the path of 2 existing gifs to point to docs/images/26/[etc] as explained here:
#348 (review)

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants