Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Aligned Participant_Status Assumption to front end. NULL => Active #1502

Merged
merged 2 commits into from
Jan 29, 2016
Merged

Aligned Participant_Status Assumption to front end. NULL => Active #1502

merged 2 commits into from
Jan 29, 2016

Conversation

gluneau
Copy link
Contributor

@gluneau gluneau commented Jan 25, 2016

We want the same behavior in the DQT as in the front end

from: pso.Description as Status
to: COALESCE(pso.Description,'Active') as Status

@gluneau gluneau added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Jan 25, 2016
@gluneau gluneau added this to the 16.04 milestone Jan 25, 2016
@codecov-io
Copy link

Current coverage is 22.18%

Merging #1502 into 16.04-dev will decrease coverage by -1.74% as of c142106

@@            16.04-dev   #1502   diff @@
=========================================
  Files             110     110       
  Stmts           18444   18250   -194
  Branches            0       0       
  Methods          1024    1024       
=========================================
- Hit              4413    4049   -364
  Partial             0       0       
- Missed          14031   14201   +170

Review entire Coverage Diff as of c142106

Powered by Codecov. Updated on successful CI builds.

samirdas added a commit that referenced this pull request Jan 29, 2016
@samirdas
Merge pull request #1502 from gluneau/dqt-participant-status-null-as-…
ee88f0a 
…active

Aligned Participant_Status Assumption to front end.  NULL => Active
@samirdas samirdas merged commit ee88f0a into aces:16.04-dev Jan 29, 2016
@christinerogers christinerogers added Add to Release Notes PR change should be highlighted in Release notes (important security, features and bugfixes) Caveat for Existing Projects PR contains changes that might impact the code or accepted practices of current active projects and removed Add to Release Notes PR change should be highlighted in Release notes (important security, features and bugfixes) Caveat for Existing Projects PR contains changes that might impact the code or accepted practices of current active projects labels May 2, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees

@driusan driusan

Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label)
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.
5 participants
@gluneau @codecov-io @driusan @samirdas @christinerogers