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

[RHCLOUD-28663] Add a query parameter to links in Events emitted by Notifications #3222

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jessicarod7
Copy link
Member

@jessicarod7 jessicarod7 commented Dec 19, 2024

Jira issue

https://issues.redhat.com/browse/RHCLOUD-28663

Description

Important

Merge after #3221 and #3229.

This builds on RHCLOUD-22813, and adds a new query parameter ?from=notifications to the two URLs generated by InsightsUrlBuilder:

  • data.inventory_url
  • data.application_url

This change will affects URLs in integrations via Camel (Slack, Microsoft Teams, Google Chat) and PagerDuty.

Testing

  • Add test cases for PagerDutyTransformer and CamelProcessorTest to ensure that the query parameter is added

@jessicarod7
Copy link
Member Author

/retest

@g-duval
Copy link
Contributor

g-duval commented Jan 7, 2025

@jessicarod7 , according Jira's comments, we could also add the integration type, I think "just" updating qute templates when they are available should be fine.
For others like PagerDuty, Splunk and Snow, may have to be hardcoded.

@jessicarod7
Copy link
Member Author

jessicarod7 commented Jan 7, 2025

Ah, I missed that comment. I'll see what I can do tomorrow.

@jessicarod7 jessicarod7 force-pushed the RHCLOUD-28663 branch 5 times, most recently from fc3aa38 to f6eee6f Compare January 10, 2025 18:00
@jessicarod7 jessicarod7 force-pushed the RHCLOUD-28663 branch 3 times, most recently from 63090e9 to ee00177 Compare January 23, 2025 22:31
@jessicarod7 jessicarod7 marked this pull request as draft January 23, 2025 23:22
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