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

Link from MS Teams notifications to report don't work #7614

Closed
fniessink opened this issue Dec 6, 2023 · 0 comments · Fixed by #7798
Closed

Link from MS Teams notifications to report don't work #7614

fniessink opened this issue Dec 6, 2023 · 0 comments · Fixed by #7798
Assignees
@fniessink
Labels
Bug Something isn't working

Comments

@fniessink
Copy link
Member

fniessink commented Dec 6, 2023 •
edited
Loading

Describe the bug
MS Teams changes the URL in the notification to the report into something that doesn't work.

To Reproduce
Steps to reproduce the behaviour:

  1. Go to a MS Team notification
  2. Click on the notification
  3. Nothing happens

The URL in the notification seems to be a URL to the notification itself instead of back to Quality-time.

Expected behaviour
Clicking the notification URL should open the Quality-time report.

Possible solution

Use msg.addLinkButton(btn_text, btn_link)

See https://github.com/rveachkc/pymsteams#add-a-link-button.

Reported by @Sebastiaan127001
@fniessink fniessink added the Bug Something isn't working label Dec 6, 2023
@fniessink fniessink mentioned this issue Dec 6, 2023
Closed
4 tasks
fniessink added a commit that referenced this issue Jan 4, 2024
@fniessink
Links in MS Teams notifications were incorrect. Note: the fix only wo…
031895a 
…rks for new notification destinations. Please recreate existing notification destinations in *Quality-time* to get correct links in MS Teams notifications. Fixes #7614.
@fniessink fniessink mentioned this issue Jan 4, 2024
Merged
@fniessink fniessink linked a pull request Jan 4, 2024 that will close this issue
Links in MS Teams notifications were incorrect. Note: the fix only wo… #7798
Merged
fniessink added a commit that referenced this issue Jan 4, 2024
@fniessink
Links in MS Teams notifications were incorrect. Note: the fix only wo…
d4fb1f6 
…rks for new notification destinations. Please recreate existing notification destinations in *Quality-time* to get correct links in MS Teams notifications. Fixes #7614.
@fniessink fniessink self-assigned this Jan 4, 2024
fniessink added a commit that referenced this issue Jan 4, 2024
@fniessink
Links in MS Teams notifications were incorrect. Note: the fix only wo…
9c46334 
…rks for new notification destinations. Please recreate existing notification destinations in *Quality-time* to get correct links in MS Teams notifications. Fixes #7614.
fniessink added a commit that referenced this issue Jan 5, 2024
@fniessink
Links in MS Teams notifications were incorrect. Note: the fix only wo…
152da76 
…rks for new notification destinations. Please recreate existing notification destinations in *Quality-time* to get correct links in MS Teams notifications. Fixes #7614.
fniessink added a commit that referenced this issue Jan 5, 2024
@fniessink
Links in MS Teams notifications were incorrect. Note: the fix only wo…
b849d89 
…rks for new notification destinations. Please recreate existing notification destinations in *Quality-time* to get correct links in MS Teams notifications. Fixes #7614.
@fniessink fniessink moved this to Released in Quality-time backlog May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fniessink fniessink

Labels
Bug Something isn't working
Projects
Quality-time backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Links in MS Teams notifications were incorrect. Note: the fix only wo… ICTU/quality-time
1 participant
@fniessink