Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Aug 14, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.3)

Can you help keep this open source service alive? 💖 Please sponsor : )

ignaciodob and others added 3 commits August 14, 2025 13:32
* docs: add flutter phone factor reference doc and example

* Update apps/docs/spec/supabase_dart_v2.yml

Co-authored-by: Chris Chinchilla <[email protected]>

* Update apps/docs/spec/supabase_js_v2.yml

Co-authored-by: Chris Chinchilla <[email protected]>

* Update apps/docs/spec/supabase_dart_v2.yml

Co-authored-by: Chris Chinchilla <[email protected]>

* Update apps/docs/spec/supabase_dart_v2.yml

Co-authored-by: Chris Chinchilla <[email protected]>

---------

Co-authored-by: Chris Chinchilla <[email protected]>
…ents (#37517)

* Refactor `StorageMenu` modals to replace deprecated patterns

* add test for `DeleteBucketModal` and update test setup

Note: Because this component uses `useParams`, it's necessary to have the dynamic route segment passed to `next-router-mock`'s `createDynamicRouteParser`. In order not to have to manually list all of these as the application grows, I added a glob utility that uses the `pages/` directory  to automatically generate an array of dynamic route paths in this case.

* add test for `EmptyBucketModal`

* add test for `EditBucketModal` and add `isNonNullable` utility function

* add test for `CreateBucketModal`

* implement requested changes

* implement visual fixes
@pull pull bot locked and limited conversation to collaborators Aug 14, 2025
@pull pull bot added the ⤵️ pull label Aug 14, 2025
@pull pull bot had a problem deploying to Studio E2E Tests August 14, 2025 22:13 Failure
@pull pull bot merged commit ca4b3bc into code:master Aug 14, 2025
0 of 9 checks passed
@pull pull bot had a problem deploying to Studio E2E Tests August 14, 2025 22:13 Error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants