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

Add refresh all visibile plots action #1808

Merged
merged 1 commit into from
Jun 1, 2022

Conversation

mattseddon
Copy link
Member

@mattseddon mattseddon commented Jun 1, 2022

@sroy as discussed here.

Feel free to leave this separate or merge in. Whatever works for you.

@mattseddon mattseddon self-assigned this Jun 1, 2022
@mattseddon mattseddon changed the base branch from main to plots-ribbon-add-exp June 1, 2022 02:54
@mattseddon mattseddon requested a review from sroy3 June 1, 2022 02:55
@mattseddon mattseddon marked this pull request as ready for review June 1, 2022 02:55
@codeclimate
Copy link

codeclimate bot commented Jun 1, 2022

Code Climate has analyzed commit e2b2799 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.8% (0.0% change).

View more on Code Climate.

@sroy3 sroy3 merged commit de2a61f into plots-ribbon-add-exp Jun 1, 2022
@sroy3 sroy3 deleted the add-refresh-all-action branch June 1, 2022 13:15
sroy3 added a commit that referenced this pull request Jun 2, 2022
* Add plots ribbon

* Fix tests and event name

* Add tests

* Add react-virtualized

* Add extension test

* Add button in plots ribbon to add experiments

* Add tests

* Add refresh all button

* add refresh all visible action to plots (#1808)

Co-authored-by: mattseddon <37993418+mattseddon@users.noreply.github.com>
sroy3 added a commit that referenced this pull request Jun 3, 2022
* Add plots ribbon

* Fix tests and event name

* Add tests

* Add react-virtualized

* Add extension test

* Add button in plots ribbon to add experiments

* Add tests

* Add refresh all button

* add refresh all visible action to plots (#1808)

* Use copy button instead of creating new copy function

* Fix tests

* Fix format after merge

Co-authored-by: mattseddon <37993418+mattseddon@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants