Skip to content

Fix 5851 issue - #6507

Closed
fogelito wants to merge 87 commits into
mainfrom
fix-5851-issue
Closed

fogelito wants to merge 87 commits into
mainfrom
fix-5851-issue

Conversation

@fogelito

@fogelito fogelito commented Oct 5, 2023

Copy link
Copy Markdown
Contributor

PineappleIOnic and others added 30 commits September 6, 2023 09:54
The project deletion failed because old code for deleting domain was
still present. By removing the old code and replacing it with the new
delete rule code, the project deletion is now working.
Prior to this, deleting a VCS deployment would fail because the
deployment path for VCS deployments are empty. Because an error gets
thrown, the delete function would stop and not delete the build files.

This change updates the worker to only delete the storage files if the
path is set.
christyjacob4 and others added 27 commits September 27, 2023 21:37
Update CHANGES.md to add fix for avatar initials
Because the tests referenced images on the appwrite.io website, but the
appwrite.io website changed, the tests broke. This updates the tests to
reference valid images/URLs.

In addition, the favicon test was changed to use github.com because
appwrite.io has an SVG favicon which the avatars API does not support.
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.

9 participants