Skip to content

Resolve Delete Endpoint Status 204 Hang Error on Frontend#43

Open
djasmit wants to merge 2 commits intoRedback-Operations:mainfrom
djasmit:Resolve-204-Status-Error
Open

Resolve Delete Endpoint Status 204 Hang Error on Frontend#43
djasmit wants to merge 2 commits intoRedback-Operations:mainfrom
djasmit:Resolve-204-Status-Error

Conversation

@djasmit
Copy link
Copy Markdown
Contributor

@djasmit djasmit commented Oct 2, 2025

Quick PR to solve the mis-formatting of the 204 return on the Delete endpoint. This issue went unnoticed as it did not impact the original Flutter app, nor did it impact the web build of the new React frontend. However the Android version of the React App rejects a body with the 204 return, causing it to not fetch the reply and staying stuck.

This quick fix resolves that by removing the one return of 204 that still contained a message body (Delete Endpoint). Has already been tested with the frontend and confirmed solution to the issue.

Quick PR to solve the mis-formatting of the 204 return on the Delete endpoint.
This issue went unnoticed as it did not impact the original Flutter app, nor did it impact the web build of the new React frontend.
However the Android version of the React App rejects a body with the 204 return, causing it to not fetch the reply and staying stuck.

This quick fix resolves that by removing the one return of 204 that still contained a message body (Delete Endpoint). Has already been tested with the frontend and confirmed solution to the issue.
@djasmit djasmit requested review from a team as code owners October 2, 2025 11:42
Mouse933
Mouse933 previously approved these changes Dec 6, 2025
Copy link
Copy Markdown

@Mouse933 Mouse933 left a comment

Choose a reason for hiding this comment

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

Hi, this looks good, no issues with the changes :)

@Mouse933 Mouse933 closed this Dec 7, 2025
@Mouse933 Mouse933 reopened this Dec 7, 2025
MorgaineBarter
MorgaineBarter previously approved these changes Feb 1, 2026
@MorgaineBarter MorgaineBarter dismissed stale reviews from Mouse933 and themself via 4970c25 February 1, 2026 22:39
Copy link
Copy Markdown

@raymondchoy14 raymondchoy14 left a comment

Choose a reason for hiding this comment

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

All good!!

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.

4 participants