Skip to content

Conversation

strmfos
Copy link
Contributor

@strmfos strmfos commented May 28, 2025

Hey team—noticed a dead link, replaced it with a working URL
https://developer.gnome.org/notification-spec/ - old link
https://developer.gnome.org/documentation/tutorials/notifications.html - new link

@DrahtBot
Copy link
Contributor

DrahtBot commented May 28, 2025

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32629.

Reviews

See the guideline for information on the review process.
A summary of reviews will appear here.

@hebasto hebasto added the GUI label May 28, 2025
@strmfos
Copy link
Contributor Author

strmfos commented May 29, 2025

@hebasto I removed the outdated link (comment), if I can do anything else, I'll be glad
Thank you

@hebasto
Copy link
Member

hebasto commented May 29, 2025

Please squash your commits and update the PR title and description.

@fanquake
Copy link
Member

I'll cherry pick this into a different branch.

@fanquake fanquake closed this May 29, 2025
fanquake added a commit that referenced this pull request Jun 3, 2025
e50312e doc: fix typos (fanquake)
c797e50 ci: update codespell to 2.4.1 (fanquake)
21ee656 doc: Remove obselete link in notificator.cpp (strmfos)
ee4406c doc: update URLs (fanquake)

Pull request description:

  A round up of #32629 + some other changes that had previously been PR'd.

ACKs for top commit:
  maflcko:
    review ACK e50312e 🥗

Tree-SHA512: 8fa3e14fdfa0cf65a42debc9cbb1f8b379aba44aa185e2e27337431e884d169bf1e811655c3a884d918e65ea28c5767ddaabaf25c862ebd9b4b38a0229ec5a93
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.

4 participants