Skip to content

refactor: [M3-7229] - Migrate UserDefinedSelect from a TextField Select to an Autocomplete#9756

Merged
bnussman-akamai merged 3 commits intolinode:developfrom
bnussman-akamai:M3-7229-migrate-UserDefinedSelect-from-textfield-to-autocomplete
Oct 12, 2023
Merged

refactor: [M3-7229] - Migrate UserDefinedSelect from a TextField Select to an Autocomplete#9756
bnussman-akamai merged 3 commits intolinode:developfrom
bnussman-akamai:M3-7229-migrate-UserDefinedSelect-from-textfield-to-autocomplete

Conversation

@bnussman-akamai
Copy link
Member

Description 📝

Preview 📷

Before After
Screenshot 2023-10-04 at 1 33 01 PM Screenshot 2023-10-04 at 1 32 21 PM

How to test 🧪

  • Test the functionality of Marketplace App Select Components
  • This is testable on the Minecraft marketplace app
  • Verify Player Idle Timeout is selectable and shows the selected value (playeridletimeout) when the Linode POST happens

@bnussman-akamai bnussman-akamai requested a review from a team as a code owner October 4, 2023 17:44
@bnussman-akamai bnussman-akamai self-assigned this Oct 4, 2023
@bnussman-akamai bnussman-akamai requested review from cliu-akamai and cpathipa and removed request for a team October 4, 2023 17:44
Copy link
Contributor

@cpathipa cpathipa left a comment

Choose a reason for hiding this comment

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

LGTM! Confirming on Marketplace App Select Components are working as expected.

image

Copy link
Contributor

@abailly-akamai abailly-akamai left a comment

Choose a reason for hiding this comment

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

Thanks!

  • No visual regression
  • No functional regression

@jaalah-akamai jaalah-akamai added the Approved Multiple approvals and ready to merge! label Oct 12, 2023
@bnussman-akamai bnussman-akamai merged commit 684e235 into linode:develop Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants