Skip to content

fix(frontend) :: remove any types from sqlpage.ts - #1515

Open
81reap wants to merge 1 commit into
81reap/ts-9-leaflet-typesfrom
81reap/ts-10-any-sqlpage
Open

81reap wants to merge 1 commit into
81reap/ts-9-leaflet-typesfrom
81reap/ts-10-any-sqlpage

Conversation

@81reap

@81reap 81reap commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

sqlpage.ts now has no implicit anys.

Fixing this smoked out two bugs ::

  1. init_bootstrap_components read event.target without knowing what kind of element. We also tighting scope from Element to HTMLElement.
  2. normalize_hash passed undefined to decodeURIComponent, which stringifies it, so a toast with no data-toast-trigger opened at #undefined.

Stack created with GitHub Stacks CLI • Give Feedback 💬

`sqlpage.ts` now has no implicit anys.

Fixing this smoked out two bugs ::
1. `init_bootstrap_components` read `event.target` without knowing what kind of element. We also tighting scope from `Element` to `HTMLElement`.
2. `normalize_hash` passed `undefined` to `decodeURIComponent`, which stringifies it, so a toast with no `data-toast-trigger` opened at `#undefined`.
@81reap
81reap added this pull request to stack #1512 September 30, 2026 00:38
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