Skip to content

RM-171: Various fixes#52

Merged
BenHall-1 merged 2 commits intoTicketsBot-cloud:masterfrom
biast12:various-fixes
Oct 24, 2025
Merged

RM-171: Various fixes#52
BenHall-1 merged 2 commits intoTicketsBot-cloud:masterfrom
biast12:various-fixes

Conversation

@biast12
Copy link
Copy Markdown
Contributor

@biast12 biast12 commented Oct 24, 2025

RM-171

Description

Fix a handful of errors (general and console errors)

  • Fix "undefined" warning
  • Fix "loadData is not defined" console error
  • Fix Bloxlink image not being shown in integrations (console error too)
  • Fix statuspage (console error)

Type of Change

  • Bug fix
  • New feature
  • Breaking change

Testing

Try to update your settings for the two first issues
Bloxlink and statuspage can not be tested as it's only an issue on live, but it's also an obvious fix

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Extracted settings loading logic into a new loadData() function in SettingsCard.svelte for improved readability and maintainability. Updated Content-Security-Policy in _headers to allow images from dbl-static.b-cdn.net and frames from j7dytlqrntrm.statuspage.io.
@biast12 biast12 requested a review from a team as a code owner October 24, 2025 12:22
@biast12 biast12 changed the title Various fixes RM-171: Various fixes Oct 24, 2025
Copy link
Copy Markdown
Member

@BenHall-1 BenHall-1 left a comment

Choose a reason for hiding this comment

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

lgtm - all errors appear to have disappered

@BenHall-1 BenHall-1 merged commit 3459547 into TicketsBot-cloud:master Oct 24, 2025
@biast12 biast12 deleted the various-fixes branch October 25, 2025 17:20
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.

2 participants