Skip to content

[pull] master from supabase:master#2

Merged
pull[bot] merged 20 commits intocode:masterfrom
supabase:master
Apr 27, 2024
Merged

[pull] master from supabase:master#2
pull[bot] merged 20 commits intocode:masterfrom
supabase:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Apr 27, 2024

See Commits and Changes for more details.


Created by pull[bot]

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

grehnen and others added 20 commits April 25, 2024 16:13
Update Swedish README

Co-authored-by: ErikSimonson <grufe@users.noreply.github.com>
* feat: add blog post about support tooling

* minor changes

* Update apps/www/_blog/2024-03-01-exploring-support-tooling.mdx

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>

* Update apps/www/_blog/2024-03-01-exploring-support-tooling.mdx

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>

---------

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>
* chore: update date of blog post

* minor update

* Update 2024-04-25-exploring-support-tooling.mdx
* nix blog post

* prettier
…and prod for native Google sign-in on Android (#23254)

Add note about sha-1 certificate for going prod on expo and Flutter docs
* Add db inspect docs link

* Update
* chore: update image versions for docker/docker-compose.yml

* chore: update image versions for docker/docker-compose.yml

* chore: update image versions for docker/docker-compose.yml
* init TreeView

* init

* update stories and type

* editing capability

* Update TreeView.tsx

* Add multiselect example

* clean

---------

Co-authored-by: Joshen Lim <joshenlimek@gmail.com>
Revert "ref: Add sentry logging to debug vercel issue (#23289)"

This reverts commit 8f6ea0c.
* blog post - postgres bloat

* Update apps/www/_blog/2024-04-26-postgres-bloat.mdx

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>

* Update apps/www/_blog/2024-04-26-postgres-bloat.mdx

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>

---------

Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>
Update the linter with the latest code.
@pull pull Bot added the ⤵️ pull label Apr 27, 2024
@pull pull Bot merged commit b2bae13 into code:master Apr 27, 2024
pull Bot pushed a commit that referenced this pull request Mar 6, 2026
…irmationDialog (supabase#43430)

## What kind of change does this PR introduce?

Form handling improvement.

## What is the current behavior?

supabase#43201 standardised our
discard changes behaviour with a shared hook and
`DiscardChangesConfirmationDialog` component. But many forms and sheets
still:

1. Don’t have any Discard-confirm close behaviour, making it too easy to
make accidental discards
2. Use a more complicated, manually-created `CloseConfirmationModal`
approach

## What is the new behavior?

- Replaced all instances of `#2` above that had `CloseConfirmationModal`
with `DiscardChangesConfirmationDialog` and its hook
- Improved design system documentation around dirty form dismissal

| Before | After |
| --- | --- |
| <img width="987" height="569" alt="Mercor Apexroles Foo
Supabase-9A40EC7C-F335-4B26-B567-450FC0845463"
src="https://github.com/user-attachments/assets/363bed82-34d2-4cc8-9164-6d18cfdbdbbc"
/> | <img width="987" height="569" alt="Mercor Apexroles Foo
Supabase-F427F1FA-DECC-4194-B663-A9E5A6F285A1"
src="https://github.com/user-attachments/assets/d49fafdc-a5c2-46df-9b67-ec42bacbe716"
/> |

## To test

Try editing values these sheets in staging, then blurring the sheet or
pressing `esc`:

- CreateQueueSheet.tsx
- CronJobsTab.tsx
- CronJobPage.tsx
- EditWrapperSheet.tsx
- OverviewTab.tsx
- WrappersTab.tsx
- CreateFunction/index.tsx
- EditHookPanel.tsx
- TriggerSheet.tsx
- SidePanelEditor.tsx
- EditSecretSheet.tsx
- PolicyEditorModal/index.tsx
- PolicyEditorPanel/index.tsx

## Still to come

- [ ] Incrementally take on `#1`: implement
`DiscardChangesConfirmationDialog` and its hook in sheets or dialog
forms that have no dirty form dismissal handling

---------

Co-authored-by: Joshen Lim <joshenlimek@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.