Skip to content

Conversation

@david-crespo
Copy link
Collaborator

Less jarring to link to the router than to an open form. Also makes it easier to get back with keyboard shortcuts because they're not eaten by the form.

image

@vercel
Copy link

vercel bot commented Dec 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
console ✅ Ready (Inspect) Visit Preview Dec 13, 2024 6:33pm

Copy link
Contributor

@charliepark charliepark left a comment

Choose a reason for hiding this comment

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

Awesome. The back button getting 86'd was frustrating; I think this'll be better in a couple of ways.

@david-crespo david-crespo merged commit c1fbf63 into main Dec 13, 2024
8 checks passed
@david-crespo david-crespo deleted the route-link branch December 13, 2024 18:41
david-crespo added a commit to oxidecomputer/omicron that referenced this pull request Dec 13, 2024
oxidecomputer/console@927c8b6...c1ebd8d

* [c1ebd8d9](oxidecomputer/console@c1ebd8d9) take inventory switches tab back out, DB is empty
* [2a139028](oxidecomputer/console@2a139028) tone down empty switches table message
* [2b0c3f12](oxidecomputer/console@2b0c3f12) oxidecomputer/console#2621
* [c1fbf631](oxidecomputer/console@c1fbf631) oxidecomputer/console#2622
* [1dd25f63](oxidecomputer/console@1dd25f63) oxidecomputer/console#2615
* [11486f8b](oxidecomputer/console@11486f8b) oxidecomputer/console#2620
* [ada302ce](oxidecomputer/console@ada302ce) oxidecomputer/console#2488
* [bc3161ae](oxidecomputer/console@bc3161ae) minor: remove stub e2e test
* [9e1d53c6](oxidecomputer/console@9e1d53c6) sentence case idp form heading
* [aaf1154f](oxidecomputer/console@aaf1154f) add extra assert for instance create with additional disks test flake
* [79d610dc](oxidecomputer/console@79d610dc) oxidecomputer/console#2618
* [bdbc02b7](oxidecomputer/console@bdbc02b7) oxidecomputer/console#2589
* [7a8ee0ab](oxidecomputer/console@7a8ee0ab) oxidecomputer/console#2614
* [0b5220a1](oxidecomputer/console@0b5220a1) npm audit fix
* [0c873cf4](oxidecomputer/console@0c873cf4) oxidecomputer/console#2610
* [d031c8ff](oxidecomputer/console@d031c8ff) bump playwright for navigation hang fix
* [dbd8545e](oxidecomputer/console@dbd8545e) oxidecomputer/console#2609
* [dc5562fe](oxidecomputer/console@dc5562fe) move error log to avoid failing to log certain errors
david-crespo added a commit to oxidecomputer/omicron that referenced this pull request Dec 14, 2024
oxidecomputer/console@927c8b6...c1ebd8d

* [c1ebd8d9](oxidecomputer/console@c1ebd8d9)
take inventory switches tab back out, DB is empty
* [2a139028](oxidecomputer/console@2a139028)
tone down empty switches table message
* [2b0c3f12](oxidecomputer/console@2b0c3f12)
oxidecomputer/console#2621
* [c1fbf631](oxidecomputer/console@c1fbf631)
oxidecomputer/console#2622
* [1dd25f63](oxidecomputer/console@1dd25f63)
oxidecomputer/console#2615
* [11486f8b](oxidecomputer/console@11486f8b)
oxidecomputer/console#2620
* [ada302ce](oxidecomputer/console@ada302ce)
oxidecomputer/console#2488
* [bc3161ae](oxidecomputer/console@bc3161ae)
minor: remove stub e2e test
* [9e1d53c6](oxidecomputer/console@9e1d53c6)
sentence case idp form heading
* [aaf1154f](oxidecomputer/console@aaf1154f)
add extra assert for instance create with additional disks test flake
* [79d610dc](oxidecomputer/console@79d610dc)
oxidecomputer/console#2618
* [bdbc02b7](oxidecomputer/console@bdbc02b7)
oxidecomputer/console#2589
* [7a8ee0ab](oxidecomputer/console@7a8ee0ab)
oxidecomputer/console#2614
* [0b5220a1](oxidecomputer/console@0b5220a1)
npm audit fix
* [0c873cf4](oxidecomputer/console@0c873cf4)
oxidecomputer/console#2610
* [d031c8ff](oxidecomputer/console@d031c8ff)
bump playwright for navigation hang fix
* [dbd8545e](oxidecomputer/console@dbd8545e)
oxidecomputer/console#2609
* [dc5562fe](oxidecomputer/console@dc5562fe)
move error log to avoid failing to log certain errors
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.

3 participants