pixiebrix-extension

pixiebrix-extension copied to clipboard

Seen in https://github.com/pixiebrix/pixiebrix-extension/pull/8016#discussion_r1535389275 Places that can benefit from the popover API, if they don't already use it: - notifications - quickbar - command popover - modals - https://github.com/pixiebrix/pixiebrix-extension/issues/8028

fregante

PLACEHOLDER: can't re-use subscribe methods across useAsyncExternalStore snapshot generators

3

twschiller

Strict null checks migration

4

I think there should be a main tracking issue for ease of reference. I'll be using this.

fregante

Review logger context usage

2

This is related to `strictNullChecks`. I'm confused about why the logging utility is used to pass data around. I wonder if this context should be accessed differently, can we find...

fregante

Discussion in https://github.com/pixiebrix/pixiebrix-extension/pull/7667#discussion_r1495953357 There are currently ~100 issues for `exactOptionalPropertyTypes`, some of which look easy to fix. I think we'd want to fix issues like this before making it stricter:...

fregante

Refactor/simplify `CreatableAutoSuggest` (too many `strictNullChecks` issues)

1

_Originally posted by @BLoe in https://github.com/pixiebrix/pixiebrix-extension/pull/6683#discussion_r1368895783_ > Furthermore, this `CreatableAutoSuggest` component appears to only be used in one place, in `SelectorSelectorWidget`. I'm not sure if this was used in more...

fregante

Incremental strictNullChecks adoption: components/formBuilder/formBuilderHelpers.ts

1

I've been skipping this file because I'm not sure of how you'd like to handle it: https://github.com/pixiebrix/pixiebrix-extension/blob/main/src/components/formBuilder/formBuilderHelpers.ts ``` [3:45:34 PM] File change detected. Starting incremental compilation... src/components/formBuilder/formBuilderHelpers.ts:185:5 - error TS18048:...

fregante

Incremental strictNullChecks adoption - Slice X (add contentScript/messenger/api.ts)

7

Follow-up of https://github.com/pixiebrix/pixiebrix-extension/issues/6498 `contentScript/messenger/api.ts` is being split off because of the 700 ts errors that need to be resolved from adding it

mnholtz

Bump @apidevtools/json-schema-ref-parser from 11.X

3

Context --- - There's a regression we have to watch out for. See thread here: https://github.com/pixiebrix/pixiebrix-extension/pull/6521 - Ignored dependabot here: https://github.com/pixiebrix/pixiebrix-extension/pull/6551

twschiller

Parts to review: - Popover itself (e.g., how to mark "selected" given that entries are buttons?) - Navigation control information

twschiller