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

Added warning log if issue link type not present #1787

Merged
merged 3 commits into from
Jan 1, 2024

Conversation

SageTheWizard
Copy link
Contributor

This error correction created problems for me, resulting in numerous issues to have inverted links (i.e. Blocks / Is Blocked By was inverted).
Debugging the issue was difficult and required to look into this code to determine what actually happened, if a log message was present, it would have simplified my debugging, so I've added it!

jira/client.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@adehad adehad left a comment

Choose a reason for hiding this comment

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

Small change, glad you got it sorted.

@studioj
Copy link
Collaborator

studioj commented Dec 27, 2023

master seems ok hopefully lets rebase and see

@studioj
Copy link
Collaborator

studioj commented Dec 27, 2023

Small change, glad you got it sorted.

requested changes?

@SageTheWizard
Copy link
Contributor Author

I see there's a failure deploying to the cloud, if this is something caused by my changes please let me know so that I can change as needed. (I'm doubtful as it's just a log print though :) )

Copy link
Collaborator

@adehad adehad left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution!

@adehad adehad merged commit 256f7cf into pycontribs:main Jan 1, 2024
10 of 11 checks passed
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.

None yet

4 participants