Skip to content

Conversation

@Lumabots
Copy link
Contributor

@Lumabots Lumabots commented Dec 4, 2025

Summary

Currently inside the on_timeout, if a message is deleted then it wil raise an error when trying to edit the message, this error is impossible to handle and so will just flood the console, lets rework this into ignoring it !

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.

Lumabots and others added 25 commits November 7, 2025 17:15
Introduces the AppInfo.edit coroutine to allow editing application settings. Updates AppInfo and related types to support new fields such as bot, flags, event webhooks, integration_types_config, and approximate_user_authorization_count. Also refactors type hints and improves handling of optional fields for better API compatibility.
Introduces the AppInfo.edit() method to allow editing application settings, including new and previously missing fields such as icon, cover_image, tags, install_params, integration_types_config, flags, event_webhooks_url, event_webhooks_status, and event_webhooks_types. Also adds related helper classes and properties for handling these fields and updates the CHANGELOG accordingly.
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Lumouille <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Lumouille <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Lumouille <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Lumouille <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Lumouille <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: Lumouille <[email protected]>
Updated the AppInfo class to only accept bytes or None for the icon and cover_image parameters, removing support for str. This change clarifies the expected types and may prevent type-related errors.
…atus and add setter for event_webhooks_enabled
Co-authored-by: Paillat <[email protected]>
Signed-off-by: Lumouille <[email protected]>
@Lumabots Lumabots requested review from a team as code owners December 4, 2025 08:10
@Lumabots Lumabots changed the title Timeout error handling useless feat: Timeout error now ignored Dec 4, 2025
Copy link
Member

@Paillat-dev Paillat-dev left a comment

Choose a reason for hiding this comment

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

@Lumabots I think you mistakenly based this off of your appinfo pr

- Added `Attachment.read_chunked` and added optional `chunksize` argument to
`Attachment.save` for retrieving attachments in chunks.
([#2956](https://github.com/Pycord-Development/pycord/pull/2956))
- Added `AppInfo.edit()` method and missing `AppInfo` fields.
Copy link
Member

Choose a reason for hiding this comment

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

I think this is a mistake

Lumabots and others added 8 commits December 5, 2025 10:19
Signed-off-by: Lumouille <[email protected]>
…nt#3009)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…t#3007)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ent#3008)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…3005)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@Lumabots Lumabots requested review from a team and Lulalaby as code owners December 5, 2025 08:19
@Lumabots
Copy link
Contributor Author

Lumabots commented Dec 5, 2025

i spent the past 30min trying to rebase this out of appifno, im just gonna delete the branch and create a new one

@Lulalaby Lulalaby closed this Dec 5, 2025
@Lulalaby
Copy link
Member

Lulalaby commented Dec 5, 2025

Man.. Luma. Why do I have to wake up to that mess

@Lumabots Lumabots deleted the timeout branch December 5, 2025 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants