Skip to content

ENG-1800 Create relation dialog closes when there is no match#1099

Merged
maparent merged 3 commits into
mainfrom
eng-1800-create-relation-dialog-closes-when-there-is-no-match
Jun 2, 2026
Merged

ENG-1800 Create relation dialog closes when there is no match#1099
maparent merged 3 commits into
mainfrom
eng-1800-create-relation-dialog-closes-when-there-is-no-match

Conversation

@maparent
Copy link
Copy Markdown
Collaborator

@maparent maparent commented Jun 1, 2026

@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 1, 2026

ENG-1800

@supabase
Copy link
Copy Markdown

supabase Bot commented Jun 1, 2026

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@maparent maparent requested a review from mdroidian June 1, 2026 11:22
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Copy link
Copy Markdown
Member

@mdroidian mdroidian left a comment

Choose a reason for hiding this comment

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

Just quickly looked at the code/video and changing the type of setResult to accept string doesn't seem right.

We use a similar pattern Result with the AutocompleteInput component in multiple places in the repo, is this the same pattern we use in those places (eg Canvas, Share via, Query Builder, etc)? Do those place also experience this same problem?

@maparent maparent requested a review from mdroidian June 2, 2026 14:16
Comment thread apps/roam/src/components/CreateRelationDialog.tsx
Copy link
Copy Markdown
Member

@mdroidian mdroidian left a comment

Choose a reason for hiding this comment

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

I would prefer if we stick to using Result, but this works as well I guess.

Also see graphite's comment.

@maparent
Copy link
Copy Markdown
Collaborator Author

maparent commented Jun 2, 2026

re using Result: Then we'd have to either modify AutoCompleteInput so it filters strings outside choices through a function that transforms strings into Results, or use my previous solution.

@maparent maparent merged commit 0a49dd9 into main Jun 2, 2026
9 checks passed
@maparent maparent deleted the eng-1800-create-relation-dialog-closes-when-there-is-no-match branch June 2, 2026 20:44
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