Skip to content

feat(notification): add Jira Service Management as a notification provider#6830

Merged
CommanderStorm merged 1 commit intolouislam:masterfrom
nuonic-digital:master
Jan 28, 2026
Merged

feat(notification): add Jira Service Management as a notification provider#6830
CommanderStorm merged 1 commit intolouislam:masterfrom
nuonic-digital:master

Conversation

@jankal
Copy link
Copy Markdown
Contributor

@jankal jankal commented Jan 28, 2026

Summary

In this pull request, the following changes are made:

Claude Opus 4.5 was used to create this PR.

Please follow this checklist to avoid unnecessary back and forth (click to expand)
  • ⚠️ If there are Breaking change (a fix or feature that alters existing functionality in a way that could cause issues) I have called them out
  • 🧠 I have disclosed any use of LLMs/AI in this contribution and reviewed all generated content.
    I understand that I am responsible for and able to explain every line of code I submit.
  • 🔍 Any UI changes adhere to visual style of this project.
  • 🛠️ I have self-reviewed and self-tested my code to ensure it works as expected.
  • 📝 I have commented my code, especially in hard-to-understand areas (e.g., using JSDoc for methods).
  • 🤖 I added or updated automated tests where appropriate.
  • 📄 Documentation updates are included (if applicable).
  • 🧰 Dependency updates are listed and explained.
  • ⚠️ CI passes and is green.

Screenshots for Visual Changes

SCR-20260128-ngjh

@github-actions
Copy link
Copy Markdown
Contributor

Hello and thanks for lending a paw to Uptime Kuma! 🐻👋
As this is your first contribution, please be sure to check out our Pull Request guidelines.
In particular: - Mark your PR as Draft while you’re still making changes - Mark it as Ready for review once it’s fully ready
If you have any design or process questions, feel free to ask them right here in this pull request - unclear documentation is a bug too.

@autofix-ci
Copy link
Copy Markdown
Contributor

autofix-ci bot commented Jan 28, 2026

Hi! I'm autofix logoautofix.ci, a bot that automatically fixes trivial issues such as code formatting in pull requests.

I would like to apply some automated changes to this pull request, but it looks like I don't have the necessary permissions to do so. To get this pull request into a mergeable state, please do one of the following two things:

  1. Allow edits by maintainers for your pull request, and then re-trigger CI (for example by pushing a new commit).
  2. Manually fix the issues identified for your pull request (see the GitHub Actions output for details on what I would like to change).

@jankal jankal changed the title feat(notification): add Jira Service Management as a notification provider WIP: feat(notification): add Jira Service Management as a notification provider Jan 28, 2026
@jankal jankal marked this pull request as draft January 28, 2026 14:39
Copy link
Copy Markdown
Collaborator

@CommanderStorm CommanderStorm left a comment

Choose a reason for hiding this comment

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

Mostly LGTM.
I have 2-3 questions about translations.

Please post the screenshots that we request in the contribution guide to ensure that you tested it working as expected.
I don't use/pay for Jira and don't want to set it up/pay for just for this PR. Hope you understand 😉

@jankal
Copy link
Copy Markdown
Contributor Author

jankal commented Jan 28, 2026

First, the screenshots @CommanderStorm

Event Screenshot
UP image
DOWN image
Certificate-expiry image
Domain-expiry ??
Testing image

@jankal
Copy link
Copy Markdown
Contributor Author

jankal commented Jan 28, 2026

@CommanderStorm as you see, I was unable to create a screenshot for domain expiry. The guide asks to use google.com and set a larger notification time. I tried, but I never got a notification.
How do I make this send a domain expiry notification?

@jankal
Copy link
Copy Markdown
Contributor Author

jankal commented Jan 28, 2026

@CommanderStorm updated the UI (mainly to put the docs link at the right place):
image

@jankal jankal changed the title WIP: feat(notification): add Jira Service Management as a notification provider feat(notification): add Jira Service Management as a notification provider Jan 28, 2026
@jankal jankal requested a review from CommanderStorm January 28, 2026 19:03
@jankal jankal marked this pull request as ready for review January 28, 2026 19:03
@github-actions github-actions bot added the pr:needs review this PR needs a review by maintainers or other community members label Jan 28, 2026
Copy link
Copy Markdown
Collaborator

@CommanderStorm CommanderStorm left a comment

Choose a reason for hiding this comment

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

Thank you, LGTM 👍🏻

@CommanderStorm CommanderStorm merged commit df8fff0 into louislam:master Jan 28, 2026
26 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

@jankal congrats on your first contribution to Uptime Kuma! 🐻
We hope you enjoy contributing to our project and look forward to seeing more of your work in the future! If you want to see your contribution in action, please see our nightly builds here.

@CommanderStorm CommanderStorm added this to the 2.1.0 milestone Jan 28, 2026
epifeny pushed a commit to epifeny/uptime-kuma that referenced this pull request Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:needs review this PR needs a review by maintainers or other community members

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update metrics' labels on saving an monitor.

2 participants