Skip to content

fix: 🐛 Complete user update methods#3103

Open
ToothyDev wants to merge 26 commits intoPycord-Development:masterfrom
ToothyDev:fix/complete-user-update-methods
Open

fix: 🐛 Complete user update methods#3103
ToothyDev wants to merge 26 commits intoPycord-Development:masterfrom
ToothyDev:fix/complete-user-update-methods

Conversation

@ToothyDev
Copy link
Contributor

@ToothyDev ToothyDev commented Feb 14, 2026

Summary

Not all user object updates triggered the on_user_update_event, see the exact attributes which were missing in #3078
Also fixes an issue where User.nameplate is broken and inaccessible

Information

  • This PR fixes an issue.
  • This PR adds something new (e.g. new method or parameters).
  • This PR is a breaking change (e.g. methods or parameters removed/renamed).
  • This PR is not a code change (e.g. documentation, README, typehinting,
    examples, ...).

Checklist

  • I have searched the open pull requests for duplicates.
  • If code changes were made then they have been tested.
    • I have updated the documentation to reflect the changes.
  • If type: ignore comments were used, a comment is also left explaining why.
  • I have updated the changelog to include these changes.

Fixes #3078
Depends on #3107

@pycord-app
Copy link

pycord-app bot commented Feb 14, 2026

Thanks for opening this pull request!
Please make sure you have read the Contributing Guidelines and Code of Conduct.

This pull request can be checked-out with:

git fetch origin pull/3103/head:pr-3103
git checkout pr-3103

This pull request can be installed with:

pip install git+https://github.com/Pycord-Development/pycord@refs/pull/3103/head

@Paillat-dev
Copy link
Member

image

@ToothyDev
Copy link
Contributor Author

ToothyDev commented Feb 14, 2026

I think I added all possible ones here for now. The missing ones are described in #3078 (comment) with a reason. accent_color might be doable as well but I wasn't sure because it would introduce potentially blocking // time-delaying code otherwise sync code.

Also, I tested all cases I could think of for now, but please do test as well in case I forgot about any

@Soheab Soheab added priority: medium Medium Priority status: in progress Work in Progess status: awaiting review Awaiting review from a maintainer API reflection Discord API isn't correctly reflected on hold labels Feb 16, 2026
@Paillat-dev Paillat-dev added this to the v2.8.0rc.1 milestone Feb 17, 2026
…ethods

# Conflicts:
#	CHANGELOG.md
#	discord/member.py
#	discord/types/user.py
#	discord/user.py
# Conflicts:
#	discord/collectibles.py
#	discord/member.py
#	discord/types/member.py
#	discord/types/user.py
@ToothyDev
Copy link
Contributor Author

Due to many attributes not being accessible or documented on the API and the nameplate being handled in #3107, this leaves just the avatar decoration data to trigger the event

plun1331
plun1331 previously approved these changes Feb 18, 2026
@Soheab Soheab removed status: in progress Work in Progess API reflection Discord API isn't correctly reflected on hold labels Feb 18, 2026
@Paillat-dev Paillat-dev added hold: testing This pull request requires further testing and removed status: awaiting review Awaiting review from a maintainer labels Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Our user update method(s) might miss some parts

5 participants

Comments