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

Add changelog entries for performance improvements, bug fixes, event support and new runtimes #1496

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ebenezerdon
Copy link
Contributor

What does this PR do?

Adds changelog entries for performance improvements, bug fixes, event support and new runtimes

Test Plan

  • Run the app and navigate to /changelog
  • Confirm that new entries are accurate and displayed correctly

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Nov 15, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-11-15 18:19:45 CET

Copy link
Contributor

@EVDOG4LIFE EVDOG4LIFE left a comment

Choose a reason for hiding this comment

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

Looks good to me, however I would recommend waiting for merging until Monday to be safe


- Fixed a bug where 409 responses during runtime creation were incorrectly treated as errors. Requests now retry until the runtime is ready.
- Fixed CDN 502 errors by refining the server compression strategy.
- Addressed S3 503 errors by adding retry logic, following S3's recommended guidelines.
Copy link
Contributor

@stnguyen90 stnguyen90 Nov 15, 2024

Choose a reason for hiding this comment

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

We should refrain from mentioning any specific provider.

Suggested change
- Addressed S3 503 errors by adding retry logic, following S3's recommended guidelines.
- Addressed storage provider 503 errors by adding retry logic, following storage provider's recommended guidelines.

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