Skip to content

fix(many): @comments overwriting scripts section#854

Merged
itinerare merged 1 commit into
lk-arpg:release/v3.0.0from
SpeedyD:fix/comments-override
Feb 26, 2024
Merged

fix(many): @comments overwriting scripts section#854
itinerare merged 1 commit into
lk-arpg:release/v3.0.0from
SpeedyD:fix/comments-override

Conversation

@SpeedyD
Copy link
Copy Markdown
Contributor

@SpeedyD SpeedyD commented Feb 26, 2024

This is not a problem in main, but only in release (and dev) because of the updated comments function.

Without this @parent, many pages which have an @comments tag will find their scripts function completely non-functional, as the scripts section in the comments page just.. overrides any other.

(For example: The admin side of handling reports, the Assign button did NOTHING.)

@itinerare
Copy link
Copy Markdown
Member

itinerare commented Feb 26, 2024

Ah.

@itinerare itinerare added bug Something isn't working needs review Pull requests that are pending community review labels Feb 26, 2024
@SpeedyD
Copy link
Copy Markdown
Contributor Author

SpeedyD commented Feb 26, 2024

Ah.

Yeah.

@itinerare itinerare added reviewed Pull requests that have received community review and are pending merge and removed needs review Pull requests that are pending community review labels Feb 26, 2024
@itinerare itinerare merged commit 8348ec3 into lk-arpg:release/v3.0.0 Feb 26, 2024
@SpeedyD SpeedyD deleted the fix/comments-override branch February 26, 2024 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working reviewed Pull requests that have received community review and are pending merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants