Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

Changes for issue #41 #77

Merged
merged 11 commits into from
Mar 15, 2017
Merged

Changes for issue #41 #77

merged 11 commits into from
Mar 15, 2017

Conversation

dlnorgaard
Copy link
Member

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling ebe364e on dnorgaard-usgs:master into d5d5edb on usgs:master.

@tparker-usgs tparker-usgs changed the base branch from master to release-2.6.3 March 10, 2017 19:45
@tparker-usgs
Copy link
Member

Combining the cleanup with your changes makes it difficult to see what changed. Can you separate those into two different commits?

@tparker-usgs
Copy link
Member

I tested the refresh button with a WWS data source by doing the following and it behaved as expected.

  1. Add channel to channel table. Click refresh and see new channel in list.
  2. Add channel to grouplinks table. Click refresh and see channel in group list.
  3. Delete channel from grouplinks table. Click refresh and see channel removed from group list, but still shown in All list.
  4. Delete channel from channel table. Click refresh and see channel removed from All list.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 9a48c80 on dnorgaard-usgs:master into d5d5edb on usgs:release-2.6.3.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 05adc7c on dnorgaard-usgs:master into d5d5edb on usgs:release-2.6.3.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling d125602 on dnorgaard-usgs:master into d5d5edb on usgs:release-2.6.3.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 0.0% when pulling 2eb187f on dnorgaard-usgs:master into d5d5edb on usgs:release-2.6.3.

@tparker-usgs tparker-usgs merged commit d91afcd into usgs:release-2.6.3 Mar 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants