Skip to content
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.

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

Remove all user account features #1338

Merged
merged 8 commits into from
Aug 22, 2024
Merged

Remove all user account features #1338

merged 8 commits into from
Aug 22, 2024

Conversation

bsian03
Copy link
Collaborator

@bsian03 bsian03 commented Feb 10, 2022

No description provided.

@coolcalcacol
Copy link
Contributor

Ain't GUILD_SYNC a legacy user implementation as well in shard?

@DonovanDMC
Copy link
Contributor

DonovanDMC commented Feb 11, 2022

you left some interfaces dangling around (UserProfile/UserSettings) and a bunch of GroupChannel remnants

And some very old bots can technically be in Group Channels iirc, I assume we aren't considering that as a valid enough reason to keep the around though (made a pr, bsian03#21)

index.d.ts Show resolved Hide resolved
index.d.ts Outdated Show resolved Hide resolved
lib/Client.js Outdated Show resolved Hide resolved
lib/gateway/Shard.js Outdated Show resolved Hide resolved
lib/gateway/Shard.js Outdated Show resolved Hide resolved
@GoldenAngel2
Copy link

Client#editGuildIntegration, Client#syncGuildIntegration, Guild#editIntegration, Guild#syncIntegration, GuildIntegration#, GuildIntegration#sync, GuildIntegration#edit can be removed as well, since those can only be used by user accounts.

@bsian03
Copy link
Collaborator Author

bsian03 commented Feb 11, 2022

I'm in the process of going through all of them now that I'm on my laptop

lib/voice/VoiceConnection.js Outdated Show resolved Hide resolved
@DonovanDMC
Copy link
Contributor

DonovanDMC commented May 9, 2022

Client#getChannel still checks for group channels, which no longer exist

various remnants of Group Channels have been left dotted around

@HeadTriXz
Copy link
Contributor

Client#getChannel still checks for group channels, which no longer exist

various remnants of Group Channels have been left dotted around

Is it worth it to still support group channels though?

@DonovanDMC
Copy link
Contributor

Client#getChannel still checks for group channels, which no longer exist
various remnants of Group Channels have been left dotted around

Is it worth it to still support group channels though?

bsian half removed them and left behind remnants of them
they aren't worth having around

Copy link
Contributor

@DonovanDMC DonovanDMC left a comment

Choose a reason for hiding this comment

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

  • setting presence does not work (missing required properties, see previous comment)
  • Client#getChannel still checks for group channels, which no longer exist
  • various remnants of GroupChannels have been left around (like Client#groupChannels jsdoc, meanwhile neither the property nor class exist)

@DonovanDMC
Copy link
Contributor

More things I've just found, the event definitions for callCreate, callDelete, callRing & callUpdate are still floating around

lib/Client.js Outdated Show resolved Hide resolved
DonovanDMC added a commit to DonovanArchive/ErisPRUpdateBot that referenced this pull request Jul 18, 2022
@gamer0mega
Copy link

you may aswell want to remove the hit property from Message, as it is meant for a user-only search feature

index.d.ts Outdated Show resolved Hide resolved
lib/Client.js Outdated Show resolved Hide resolved
Copy link
Contributor

@DonovanDMC DonovanDMC left a comment

Choose a reason for hiding this comment

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

besides the above, LGTM

@DonovanDMC
Copy link
Contributor

DonovanDMC commented Aug 8, 2022

I've gone back through some documentation, and now see that GroupChannels are not purely userbot features. Any removals or changes should be undone. Correction: The removals from they gateway are fine, all the other removals should be undone.

#1406 properly implements the official api methods for them.

@HeadTriXz
Copy link
Contributor

I've gone back through some documentation, and now see that GroupChannels are not purely userbot features. Any removals or changes should be undone.

#1406 properly implements the official api methods for them.

Bots can not join group channels anymore though, some bots may still be in one because it was possible in the past. But they do not receive any messages.

@DonovanDMC
Copy link
Contributor

I didn't say they could join them, they can't be received over the gateway sure but they can be retrieved from rest just fine if the bot created it

@bsian03
Copy link
Collaborator Author

bsian03 commented Aug 8, 2022

Don't think it's worth maintaining/keeping in Eris. The % of bots having continuous access to group channels is miniscule, if anything
Removed gateway references

@bsian03 bsian03 marked this pull request as draft January 10, 2024 01:57
@bsian03 bsian03 mentioned this pull request Jan 17, 2024
54 tasks
@bsian03
Copy link
Collaborator Author

bsian03 commented Jul 20, 2024

bsian03/eris@f4db90e...5168831 for reference

@bsian03 bsian03 marked this pull request as ready for review July 25, 2024 14:10
Copy link
Contributor

@TTtie TTtie left a comment

Choose a reason for hiding this comment

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

took a very, very quick glance over this, here's what I've found

index.d.ts Show resolved Hide resolved
index.d.ts Outdated Show resolved Hide resolved
lib/gateway/Shard.js Outdated Show resolved Hide resolved
lib/gateway/Shard.js Outdated Show resolved Hide resolved
Copy link
Contributor

@DonovanDMC DonovanDMC left a comment

Choose a reason for hiding this comment

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

Getting the call property is still possible in some capacity:
discord/discord-api-docs#6864

@bsian03 bsian03 removed the postponed label Aug 22, 2024
@bsian03 bsian03 merged commit 0b3ef10 into abalabahaha:dev Aug 22, 2024
flav-code added a commit to Flavibot/eris that referenced this pull request Aug 22, 2024
breaking: Remove all user account features (abalabahaha#1338)
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.

8 participants