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

Correctly identify ilmSessions which can be published #1726

Merged
merged 1 commit into from
May 26, 2016

Conversation

jrjohnson
Copy link
Member

We were depending on a key which doesn't exist. Depend on
ilmSession.dueDate to ensure that publishable ILMs show up that way.

Fixes #1658

We were depending on a key which doesn't exist.  Depend on
ilmSession.dueDate to ensure that publishable ILMs show up that way.
@jrjohnson
Copy link
Member Author

@dartajax this is ready to review to ensure you don't see any other weirdness with what is supposed to show up where in publish all.

@jrjohnson jrjohnson assigned saschaben and unassigned dartajax May 25, 2016
@saschaben
Copy link
Member

works for me 👍

@saschaben saschaben assigned jrjohnson and unassigned saschaben May 25, 2016
@jrjohnson jrjohnson assigned stopfstedt and unassigned jrjohnson May 25, 2016
@stopfstedt stopfstedt merged commit 4c0069f into ilios:master May 26, 2016
@jrjohnson jrjohnson deleted the 1658-publishall branch May 26, 2016 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants