Skip to content

fix: Added member data to the raw_reaction_remove event #2412

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
Merged
merged 8 commits into from
Apr 15, 2024
Merged

fix: Added member data to the raw_reaction_remove event #2412

merged 8 commits into from
Apr 15, 2024

Conversation

@jordan-day
Copy link
Contributor

@jordan-day jordan-day commented Mar 31, 2024

Summary

This Pull Request adds the member data to the raw_reaction_remove event
This ensures user consistency between the add/remove reaction events.

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.
    • [N/A] 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.

@jordan-day jordan-day requested a review from a team as a code owner March 31, 2024 18:17
@pullapprove4 pullapprove4 bot requested a review from BobDotCom March 31, 2024 18:17
@Lulalaby Lulalaby requested review from Dorukyum and removed request for BobDotCom April 1, 2024 14:24
@jordan-day
Copy link
Contributor Author

jordan-day commented Apr 2, 2024

ping @Dorukyum

@Lulalaby
Copy link
Member

Lulalaby commented Apr 2, 2024

Please do not ping our team, I'm goin through and ping / DM people if needed.

@JustaSqu1d JustaSqu1d added the hold: changelog This pull request is missing a changelog entry label Apr 9, 2024
@jordan-day
Copy link
Contributor Author

jordan-day commented Apr 10, 2024

I'd really like to get this reviewed merged if possible.
It is a minor change and will unblock my application

plun1331
plun1331 previously requested changes Apr 10, 2024
View reviewed changes
Copy link
Member

@plun1331 plun1331 left a comment

Choose a reason for hiding this comment

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

Changelog needed

@plun1331 plun1331 added priority: low Low Priority status: awaiting review Awaiting review from a maintainer feature Implements a feature labels Apr 10, 2024
@plun1331 plun1331 added this to the v2.6 milestone Apr 10, 2024
@jordan-day
Copy link
Contributor Author

jordan-day commented Apr 12, 2024

Changelog needed

I added the changelog update.

@plun1331 plun1331 removed the hold: changelog This pull request is missing a changelog entry label Apr 12, 2024
plun1331
plun1331 reviewed Apr 12, 2024
View reviewed changes
@plun1331
Update CHANGELOG.md
5904a87 
Signed-off-by: plun1331 <[email protected]>
JustaSqu1d
JustaSqu1d requested changes Apr 12, 2024
View reviewed changes
Copy link
Member

@JustaSqu1d JustaSqu1d left a comment

Choose a reason for hiding this comment

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

Please update the RawReactionActionEvent.member documentation accordingly with this change.

@jordan-day
Copy link
Contributor Author

jordan-day commented Apr 13, 2024

Please update the RawReactionActionEvent.member documentation accordingly with this change.

Done

@Lulalaby Lulalaby enabled auto-merge (squash) April 14, 2024 05:08
@Lulalaby Lulalaby requested a review from plun1331 April 14, 2024 05:09
@jordan-day jordan-day removed their assignment Apr 14, 2024
Dorukyum
Dorukyum approved these changes Apr 15, 2024
View reviewed changes
Copy link
Member

@Dorukyum Dorukyum left a comment

Choose a reason for hiding this comment

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

LGTM

@Lulalaby Lulalaby dismissed plun1331’s stale review April 15, 2024 11:08

done, merge the shit

Lulalaby
Lulalaby approved these changes Apr 15, 2024
View reviewed changes
Copy link
Member

@Lulalaby Lulalaby left a comment

Choose a reason for hiding this comment

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

pullapprove do your job

@Lulalaby Lulalaby removed the request for review from plun1331 April 15, 2024 11:09
@Lulalaby Lulalaby merged commit c10022c into Pycord-Development:master Apr 15, 2024
plun1331 added a commit to SkyKings-Guild/pycord that referenced this pull request Apr 21, 2024
@plun1331 @NeloBlivion
@pre-commit-ci @Lulalaby @dependabot @jordan-day @Dorukyum @JustaSqu1d
j (#4)
51be093 
* feat: add once kwarg to Cog.listener (Pycord-Development#2403)

* add once kwarg to Cog.listener

* clarify

* style(pre-commit): auto fixes from pre-commit.com hooks

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Lala Sabathil <[email protected]>

* chore(pre-commit): pre-commit autoupdate (Pycord-Development#2410)

updates:
- [github.com/asottile/pyupgrade: v3.15.1 → v3.15.2](asottile/pyupgrade@v3.15.1...v3.15.2)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* fix: pass proper state to interaction._guild (Pycord-Development#2411)

* fix: option and BridgeOption adjustments (Pycord-Development#2417)

* chore(pre-commit): pre-commit autoupdate (Pycord-Development#2416)

updates:
- [github.com/pre-commit/pre-commit-hooks: v4.5.0 → v4.6.0](pre-commit/pre-commit-hooks@v4.5.0...v4.6.0)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* feat: implement Guild.search_members (Pycord-Development#2418)

* implement guild member search

* clarification.

* style(pre-commit): auto fixes from pre-commit.com hooks

* cl

* undo iteration

* style(pre-commit): auto fixes from pre-commit.com hooks

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* ci(deps): bump ribtoks/tdg-github-action from 0.4.7.pre.beta to 0.4.10.pre.beta (Pycord-Development#2420)

ci(deps): bump ribtoks/tdg-github-action

Bumps [ribtoks/tdg-github-action](https://github.com/ribtoks/tdg-github-action) from 0.4.7.pre.beta to 0.4.10.pre.beta.
- [Release notes](https://github.com/ribtoks/tdg-github-action/releases)
- [Commits](ribtoks/tdg-github-action@v0.4.7-beta...v0.4.10-beta)

---
updated-dependencies:
- dependency-name: ribtoks/tdg-github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: plun1331 <[email protected]>

* fix: Added member data to the raw_reaction_remove event (Pycord-Development#2412)

* fix: Added member data to the raw_reaction_remove event

* style(pre-commit): auto fixes from pre-commit.com hooks

* Added changelog.md for adding member data to raw reaction remove

* Update CHANGELOG.md

Signed-off-by: plun1331 <[email protected]>

* doc: Updated documentation for the RawReactionActionEvent.member field

---------

Signed-off-by: plun1331 <[email protected]>
Co-authored-by: root <root@DESKTOP-U7KGUE3>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: plun1331 <[email protected]>

* chore(pre-commit): pre-commit autoupdate (Pycord-Development#2422)

updates:
- [github.com/psf/black: 24.3.0 → 24.4.0](psf/black@24.3.0...24.4.0)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* docs: remove false optional (Pycord-Development#2424)

docs: not optional

* fix: Guild.query_members may accept empty query and limit (Pycord-Development#2419)

Signed-off-by: UK <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: JustaSqu1d <[email protected]>
Co-authored-by: Dorukyum <[email protected]>
Co-authored-by: plun1331 <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Signed-off-by: plun1331 <[email protected]>
Signed-off-by: UK <[email protected]>
Co-authored-by: UK <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Lala Sabathil <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: jordan-day <[email protected]>
Co-authored-by: root <root@DESKTOP-U7KGUE3>
Co-authored-by: Dorukyum <[email protected]>
Co-authored-by: JustaSqu1d <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@plun1331 plun1331 plun1331 left review comments

@JustaSqu1d JustaSqu1d JustaSqu1d requested changes

@Lulalaby Lulalaby Lulalaby approved these changes

@Dorukyum Dorukyum Dorukyum approved these changes

+1 more reviewer

@Blue-Robin-Taken Blue-Robin-Taken Blue-Robin-Taken approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

feature Implements a feature priority: low Low Priority status: awaiting review Awaiting review from a maintainer

Projects

None yet

Milestone

v2.6

Development

Successfully merging this pull request may close these issues.

6 participants

@jordan-day @Lulalaby @plun1331 @Dorukyum @Blue-Robin-Taken @JustaSqu1d