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.

Sign up for GitHub

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

Jump to bottom

Feature: Clear app's discovered robots cache #2435

Closed
2 tasks
mcous opened this issue Oct 5, 2018 · 0 comments · Fixed by #5629
Closed
2 tasks

Feature: Clear app's discovered robots cache #2435

mcous opened this issue Oct 5, 2018 · 0 comments · Fixed by #5629
Labels
app Affects the `app` project feature-request A request for a new feature or a change that isn't a bug. May require further triage or scoping. robot-svcs Falls under the purview of the Robot Services squad (formerly CPX, Core Platform Experience).
Milestone
CPX Sprint 8

Comments

@mcous
Copy link
Contributor

mcous commented Oct 5, 2018 •
edited
Loading

overview

As a user that interacts with many robots (e.g. an app user at the factory), I would like to tell my app to clear out its entire discovery cache so my robots list is not polluted with a bunch of robots I don't care about.

Related to #2408

current behavior

Every robot the app ever sees is saved forever (or, until the discovery.json file in the app's data directory is deleted).

acceptance criteria

In the apps advanced settings card:

  • The user can click a button to clear the saved robot list
  • After clicking the button, the user is presented a confirmation modal. Clicking "confirm" should:
    • Wipe out discovery.json in the app's config directory
    • Restart the discovery client
@mcous mcous added feature Ticket is a feature request / PR introduces a feature app Affects the `app` project labels Oct 5, 2018
@mcous mcous mentioned this issue Oct 5, 2018
Closed
@mcous mcous added robot-svcs Falls under the purview of the Robot Services squad (formerly CPX, Core Platform Experience). WIP labels Jan 16, 2020
@mcous mcous removed this from the Connectivity: Disconnect from Network milestone Jan 24, 2020
@mcous mcous removed the WIP label Jan 24, 2020
@mcous mcous changed the title Robot Discovery: Clear/disable seen robots cache Feature: Clear app's discovered robots cache Jan 24, 2020
@mcous mcous mentioned this issue Jan 24, 2020
Closed
3 tasks
@mcous mcous added feature-request A request for a new feature or a change that isn't a bug. May require further triage or scoping. and removed feature Ticket is a feature request / PR introduces a feature labels Apr 14, 2020
@mcous mcous added this to the CPX Sprint 7 milestone Apr 14, 2020
@mcous mcous modified the milestones: CPX Sprint 7, CPX Sprint 8 Apr 28, 2020
@SyntaxColoring SyntaxColoring mentioned this issue May 2, 2020
Closed
sanni-t added a commit that referenced this issue May 5, 2020
@sanni-t
feat(app): add ability to clear cached robots
c126508 
clears discovery.json

Closes #2435
@sanni-t sanni-t mentioned this issue May 6, 2020
Merged
sanni-t added a commit that referenced this issue May 8, 2020
@sanni-t
feat(app): add UI to clear cached robots
5388d2b 
Closes #2435
sanni-t added a commit that referenced this issue May 8, 2020
@sanni-t
refactor(app): add action to clear cached robots (#5584)
c114f39 
Part of #2435
@sanni-t sanni-t mentioned this issue May 8, 2020
Merged
sanni-t added a commit that referenced this issue May 8, 2020
@sanni-t
feat(app): add UI to clear cached robots (#5629)
330acc0 
Closes #2435
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
app Affects the `app` project feature-request A request for a new feature or a change that isn't a bug. May require further triage or scoping. robot-svcs Falls under the purview of the Robot Services squad (formerly CPX, Core Platform Experience).
Projects
None yet
Milestone
CPX Sprint 8
Development

Successfully merging a pull request may close this issue.

feat(app): add UI to clear cached robots Opentrons/opentrons
1 participant
@mcous