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.

Sign up for GitHub

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

Jump to bottom

Allow you to set the reporter as well, not just the assignee #987

Merged
merged 1 commit into from
May 18, 2021
Merged

Allow you to set the reporter as well, not just the assignee #987

merged 1 commit into from
May 18, 2021

Conversation

tlianza
Copy link
Contributor

@tlianza tlianza commented Dec 9, 2020 •
edited by ssbarnea
Loading

Fixes: #404

@tlianza
Copy link
Contributor Author

tlianza commented Dec 9, 2020

@ssbarnea - any ideas on the CI error? I think this is a good fix, at least when I run it locally it resolves my issue of setting the reporter.

@adehad adehad added the bug label May 2, 2021
@adehad adehad mentioned this pull request May 2, 2021
Closed
@studioj studioj closed this May 8, 2021
@studioj studioj reopened this May 8, 2021
adehad
adehad approved these changes May 14, 2021
View reviewed changes
Copy link
Contributor

@adehad adehad left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Only legacy Travis CI failing, happy for this to get merged @ssbarnea

@adehad adehad linked an issue May 14, 2021 that may be closed by this pull request
add_user_to_group 404 error #975
Closed
@adehad adehad removed a link to an issue May 14, 2021
add_user_to_group 404 error #975
Closed
@adehad adehad linked an issue May 14, 2021 that may be closed by this pull request
Cannot set the reporter of a ticket (automatically defaults to 'assignee') #404
Closed
@ssbarnea ssbarnea closed this May 15, 2021
@ssbarnea ssbarnea reopened this May 15, 2021
@ssbarnea ssbarnea closed this May 15, 2021
@ssbarnea ssbarnea reopened this May 15, 2021
@studioj
Copy link
Collaborator

studioj commented May 15, 2021

I would add a CI test for this @tlianza.
But I'm a QA person which is a nag for those things :-)

@tlianza
Copy link
Contributor Author

tlianza commented May 17, 2021

I would add a CI test for this @tlianza.
But I'm a QA person which is a nag for those things :-)

Is there an example I can follow?

@ssbarnea ssbarnea changed the title Fixes #404 - Allow you to set the reporter as well, not just the assignee Allow you to set the reporter as well, not just the assignee May 18, 2021
@ssbarnea ssbarnea merged commit de73cfd into pycontribs:master May 18, 2021
svermeulen pushed a commit to svermeulen/jira that referenced this pull request Oct 31, 2021
@tlianza @svermeulen
Fixes pycontribs#404 - Allow you to set the reporter as well, not jus…
450ea84 
…t the assignee. (pycontribs#987)

Co-authored-by: Tom Lianza <tlianza@cloudflare.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adehad adehad adehad approved these changes

@ssbarnea ssbarnea ssbarnea approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot set the reporter of a ticket (automatically defaults to 'assignee')
4 participants
@tlianza @studioj @ssbarnea @adehad