Skip to content

Added support for state_url, details_url, large_url, small_url.#264

Merged
kaneryu merged 6 commits intoqwertyquerty:masterfrom
zRitsu:master
Oct 17, 2025
Merged

Added support for state_url, details_url, large_url, small_url.#264
kaneryu merged 6 commits intoqwertyquerty:masterfrom
zRitsu:master

Conversation

@zRitsu
Copy link
Contributor

@zRitsu zRitsu commented Oct 7, 2025

Description

This pull request introduces support for several new URL fields in the activity payload.

Adds the ability to set clickable URLs for:

  • The details text (details_url)
  • The state text (state_url)
  • The large asset image (large_url)
  • The small asset image (small_url)

References

These changes are based on the official Discord Developer Documentation:

@kaneryu kaneryu added this to the 4.7.0 milestone Oct 15, 2025
@kaneryu kaneryu added the enhancement New feature or request label Oct 15, 2025
@kaneryu
Copy link
Collaborator

kaneryu commented Oct 15, 2025

Thanks for the work! I'll look over this now. In the meantime, can you please update the commit names to use conventional commits? In this case, just amend Add support for state_url, details_url, large_url, small_url. to feat: Add support for state_url, details_url, large_url, small_url.

@kaneryu kaneryu requested review from Copilot and kaneryu October 15, 2025 18:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kaneryu kaneryu requested a review from Copilot October 15, 2025 19:27
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@zRitsu
Copy link
Contributor Author

zRitsu commented Oct 16, 2025

Okay, no problem, and thanks for the guide. 😄

I'll amend commit now with name you recommended. 👍

@kaneryu
Copy link
Collaborator

kaneryu commented Oct 17, 2025

Thanks for the change, I'm away from my computer for about 2 days, so I'll start reviewing and hopefully merge then.
(nevermind, i got the wifi working, lol)

@kaneryu kaneryu merged commit 864a081 into qwertyquerty:master Oct 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments