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

Fix/work package queries #437

Merged
merged 6 commits into from
Sep 18, 2013
Merged

Fix/work package queries #437

merged 6 commits into from
Sep 18, 2013

Conversation

GarstgerUnhold
Copy link
Contributor

Currently, saving a work package query is broken. This is an impediment for https://www.openproject.org/issues/1967. The corresponding issue is https://www.openproject.org/issues/2078.

# See doc/COPYRIGHT.rdoc for more details.
#++

Feature: Watch issues
Copy link
Contributor

Choose a reason for hiding this comment

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

This is named incorrect

@ulferts
Copy link
Contributor

ulferts commented Sep 17, 2013

Thanks for adding that many tests, really nice of you.

Just assuming: Just in case you copied over the background - then there might be the chance to remove some of the steps in there. That would speed up the execution.

A changelog entry is missing.

@GarstgerUnhold
Copy link
Contributor Author

The Background was initially copied, changed it around a bit tho. Creating an issue and the additional users could go into the grouping resp. permission scenarios, might make things more clear as to why they are there.

@hschink
Copy link
Contributor

hschink commented Sep 18, 2013

Fine on CI: I'm going to merge this! Thanks! 🌸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants