Skip to content

A better fix for #109#112

Open
edgeinfinity1 wants to merge 17 commits intoFriendsOfFlarum:1.xfrom
edgeinfinity1:1.x
Open

A better fix for #109#112
edgeinfinity1 wants to merge 17 commits intoFriendsOfFlarum:1.xfrom
edgeinfinity1:1.x

Conversation

@edgeinfinity1
Copy link
Copy Markdown
Contributor

Fixes #0000

Improved fixes in #109:

  • Now ignored errors no longer trigger the red popup frame.
  • Now save status shows a cross when saving failed.

Changes proposed in this pull request:

Reviewers should focus on:

Screenshot

Confirmed

  • Frontend changes: tested on a local Flarum installation.
  • Backend changes: tests are green (run composer test).

Required changes:

@edgeinfinity1 edgeinfinity1 requested a review from a team as a code owner December 22, 2025 06:52
@edgeinfinity1
Copy link
Copy Markdown
Contributor Author

Added fix: Fix reply drafts considered as discussions if their corresponding discussions are not loaded into app.store.

@edgeinfinity1
Copy link
Copy Markdown
Contributor Author

edgeinfinity1 commented Mar 21, 2026

Well I'm sorry that codex commited changes on compiled files that should not be included possibly.
Please tell me if I should open another PR on a clean branch.

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