Skip to content

feat: Add avatar_url override for webhook struct, resolves #1416#1419

Merged
braindigitalis merged 4 commits intobrainboxdotcc:devfrom
Enovale:avatar_url_property
Apr 20, 2025
Merged

feat: Add avatar_url override for webhook struct, resolves #1416#1419
braindigitalis merged 4 commits intobrainboxdotcc:devfrom
Enovale:avatar_url_property

Conversation

@Enovale
Copy link
Contributor

@Enovale Enovale commented Apr 18, 2025

This is a fix for #1416, adding a new property to the webhook struct to allow for custom avatar urls.

If an avatar_url is set, it will be used instead of the custom uploaded avatar.

Code change checklist

  • I have ensured that all methods and functions are fully documented using doxygen style comments.
  • My code follows the coding style guide.
  • I tested that my change works before raising the PR.
  • I have ensured that I did not break any existing API calls.
  • I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

@CLAassistant
Copy link

CLAassistant commented Apr 18, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added documentation Improvements or additions to documentation code Improvements or additions to code. labels Apr 18, 2025
@netlify
Copy link

netlify bot commented Apr 18, 2025

Deploy Preview for dpp-dev ready!

Name Link
🔨 Latest commit 2197bf2
🔍 Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/680370dce0146e000885dcd7
😎 Deploy Preview https://deploy-preview-1419--dpp-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@braindigitalis
Copy link
Contributor

just a couple of minor comment changes, and this is good to merge!
Thanks for your contribution!

@braindigitalis braindigitalis linked an issue Apr 18, 2025 that may be closed by this pull request
@Enovale Enovale requested a review from braindigitalis April 18, 2025 17:07
@braindigitalis braindigitalis merged commit 34d6b01 into brainboxdotcc:dev Apr 20, 2025
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code Improvements or additions to code. documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No way to arbitrarily override avatar_url for webhooks

4 participants