Skip to content
This repository was archived by the owner on Sep 6, 2024. It is now read-only.

Conversation

@xx-db
Copy link

@xx-db xx-db commented Aug 8, 2024

edit modal is made to support more fields in the future if we want to edit more than just description

Edit.Catalog.Description.mov

closes: #38

edit modal is made to support more fields in the future if we want to edit more than just description
Copy link
Collaborator

@jamieknight-db jamieknight-db left a comment

Choose a reason for hiding this comment

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

Looks good! Can you include a screen recording so we can see the magic in action? Also, the linter has some suggestions for you ;)

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>
@jamieknight-db jamieknight-db merged commit 6ef82e2 into unitycatalog:main Aug 12, 2024
rtyler pushed a commit to rtyler/unitycatalog that referenced this pull request Sep 6, 2024
…og-ui#63)

* Add ability to edit catalog description

edit modal is made to support more fields in the future if we want to edit more than just description

* lint

* feedback

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>

---------

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>
rtyler pushed a commit to rtyler/unitycatalog that referenced this pull request Sep 6, 2024
…og-ui#63)

* Add ability to edit catalog description

edit modal is made to support more fields in the future if we want to edit more than just description

* lint

* feedback

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>

---------

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>
dennyglee pushed a commit to unitycatalog/unitycatalog that referenced this pull request Sep 6, 2024
…og-ui#63)

* Add ability to edit catalog description

edit modal is made to support more fields in the future if we want to edit more than just description

* lint

* feedback

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>

---------

Co-authored-by: Shawn Chen <ethanloatrchen@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update catalog description

4 participants