Skip to content

fix(catalog,web): resolve SSE domain event broadcasting and state hyd… - #1348

Open
Ahmedali1010 wants to merge 1 commit into
fullstackhero:mainfrom
Ahmedali1010:fix/sse-state-hydration
Open

fix(catalog,web): resolve SSE domain event broadcasting and state hyd…#1348
Ahmedali1010 wants to merge 1 commit into
fullstackhero:mainfrom
Ahmedali1010:fix/sse-state-hydration

Conversation

@Ahmedali1010

Copy link
Copy Markdown

…ration data lossHello team,

I have identified an architectural flaw regarding SSE domain event broadcasting and React state hydration/persistence in the starter kit. I have developed a complete fix for it.

  • Backend: Implemented missing Update and Delete domain events and queued them properly for MediatR to intercept and broadcast via SSE.
  • Frontend: Implemented sessionStorage in the SSE context to resolve DOM hydration errors and prevent volatile state data loss during page refresh.

Since the private security advisory endpoint returned a 404, I am submitting this architectural fix as a standard PR for your review.

@Ahmedali1010

Copy link
Copy Markdown
Author

Hi iammukeshm (or repository maintainers),

Just a gentle follow-up on this PR. It appears that the GitHub Actions workflows are currently pending maintainer approval to execute.

Could you please approve the workflow runs when you have a moment? I am available to provide any additional technical context regarding the SSE domain event broadcasting fix or the security advisory context if needed.

Thank you for your time and efforts on this project."

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.

1 participant