Skip to content

Conversation

ryanzhangofficial
Copy link
Member

@ryanzhangofficial ryanzhangofficial commented Jun 17, 2025

Description

Adds helpful tooltips to the Robots and Fields sections of Configure Assets panel for a more intuitive UI.

Screenshot 2025-06-17 at 11 21 00 AM Screenshot 2025-06-17 at 11 21 23 AM

@ryanzhangofficial ryanzhangofficial requested review from HunterBarclay and a team as code owners June 17, 2025 17:26
@ryanzhangofficial ryanzhangofficial requested review from Dhruv-0-Arora and azaleacolburn and removed request for a team June 17, 2025 17:26
@BrandonPacewic BrandonPacewic added documentation ui/ux Relating to user interface, or in general, user experience labels Jun 17, 2025
@BrandonPacewic BrandonPacewic self-requested a review June 17, 2025 17:37
Copy link
Member

@BrandonPacewic BrandonPacewic left a comment

Choose a reason for hiding this comment

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

Looks pretty good, can you try and run the auto formatter so the workflow passes? Just looking for eslint to pass.

@ryanzhangofficial
Copy link
Member Author

Just ran it through the formatter.

Copy link
Collaborator

@Dhruv-0-Arora Dhruv-0-Arora left a comment

Choose a reason for hiding this comment

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

Could you change the branch you are requesting the PR to be merged into to be Autodesk:dev (since we don't merge directly into prod)

@ryanzhangofficial ryanzhangofficial changed the base branch from prod to dev June 17, 2025 18:08
Copy link
Collaborator

@Dhruv-0-Arora Dhruv-0-Arora left a comment

Choose a reason for hiding this comment

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

Can you remove the extra package-lock.json here

…el.tsx

Co-authored-by: Dhruv Arora <a_dhruv@outlook.com>
Copy link
Collaborator

@Dhruv-0-Arora Dhruv-0-Arora left a comment

Choose a reason for hiding this comment

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

LGTM!

@Dhruv-0-Arora Dhruv-0-Arora requested review from BrandonPacewic and removed request for HunterBarclay and azaleacolburn June 17, 2025 18:22
Copy link
Member

@BrandonPacewic BrandonPacewic left a comment

Choose a reason for hiding this comment

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

Cool looks good 🚀

@BrandonPacewic BrandonPacewic merged commit d58281f into Autodesk:dev Jun 18, 2025
13 of 14 checks passed
@BrandonPacewic BrandonPacewic mentioned this pull request Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation ui/ux Relating to user interface, or in general, user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants