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

[BUG] Stale Export Data #19

Merged
merged 1 commit into from
Sep 29, 2016

Conversation

lasley
Copy link
Member

@lasley lasley commented Sep 27, 2016

This resolves session handling in backend update method

  • Fix update session handling
  • Add direct query return to read method

@lasley lasley added this to the 9.0 milestone Sep 27, 2016
@lasley lasley self-assigned this Sep 27, 2016
@lasley lasley force-pushed the bugfix/9.0/SD-1770-stale-export-info branch 4 times, most recently from 7427bbe to eba8178 Compare September 27, 2016 17:25
* Fix update session handling
* Add direct query return to read method
@lasley lasley force-pushed the bugfix/9.0/SD-1770-stale-export-info branch from eba8178 to 6eff7e8 Compare September 27, 2016 17:33
@coveralls
Copy link

coveralls commented Sep 27, 2016

Coverage Status

Coverage increased (+0.1%) to 92.504% when pulling eba8178 on bugfix/9.0/SD-1770-stale-export-info into 90ce4ba on release/9.0.

@lasley
Copy link
Member Author

lasley commented Sep 27, 2016

Ready for review

@coveralls
Copy link

coveralls commented Sep 27, 2016

Coverage Status

Coverage increased (+0.1%) to 92.504% when pulling 6eff7e8 on bugfix/9.0/SD-1770-stale-export-info into 90ce4ba on release/9.0.

Copy link
Contributor

@tedsalmon tedsalmon left a comment

Choose a reason for hiding this comment

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

👍

@tedsalmon tedsalmon merged commit 2bf0273 into release/9.0 Sep 29, 2016
@tedsalmon tedsalmon deleted the bugfix/9.0/SD-1770-stale-export-info branch September 29, 2016 05:56
This pull request was closed.
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.

3 participants