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

Move from participants to JQL Tricks commentedByUser() #287

Merged
merged 2 commits into from
Dec 19, 2023

Conversation

stronk7
Copy link
Member

@stronk7 stronk7 commented Dec 19, 2023

The original query was working wrongly sometimes, because of Jira re-indexing problems. And that was leading to some ToBiC jobs to be relaunched, when they should not.

We have confirmed that, after a re-index, the original query is back to work, but that's not nice-enough outcome.

So we have moved to the already installed JQL Tricks query that seems to be immune to the indexing problems and properly keeps out to all the issues where ToBiC already has participated (by adding a comment).

Note that it has been checked that the JQL Tricks extension is also available for Jira Cloud (with a change in the syntax), but will work there, so it's ok if we add this new use here.

Ref: https://www.j-tricks.com/jql-tricks-cloud-migration.html

The original query was working wrongly sometimes, because of
Jira re-indexing problems. And that was leading to some ToBiC
jobs to be relaunched, when they should not.

We have confirmed that, after a re-index, the original query
is back to work, but that's not nice-enough outcome.

So we have moved to the already installed JQL Tricks query
that seems to be immune to the indexing problems and properly
keeps out to all the issues where ToBiC already has participated
(by adding a comment).

Note that it has been checked that the JQL Tricks extension is
also available for Jira Cloud (with a change in the syntax), but
will work there, so it's ok if we add this new use here.

Ref: https://www.j-tricks.com/jql-tricks-cloud-migration.html
@lameze lameze merged commit 02e74ff into moodlehq:master Dec 19, 2023
4 checks passed
@stronk7 stronk7 deleted the avoid_tobic_dupes branch December 19, 2023 12:04
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.

2 participants