Migrate components to TS: ButtonLink

[georgewrmarshall]

Description

Migrate ButtonLink component to TypeScript

Dependencies

• Create TS version of Box component in component-library #19239

Technical Details

• Convert Objects in SNAKE_CASE to enums in PascalCase (name and keys)
• Ensure enums use singular not plural e.g. Size not Sizes
• Update file name from component-name.contants.js to component-name.types.ts and add types
• Update file name from component-name.js to component-name.tsx and add types
• Update file name from component-name.test.js to component-name.test.tsx
• Update file name from component-name.stories.js to component-name.stories.tsx
• Update all instances of component objects to enums
• Update all documentation

Acceptance Criteria

• Component has been converted to TypeScript
• Stories, documentation and tests have been updated inside of the component-name folder
• Before/After screen cast of component stories in storybook to show no visual regression after migration

Difficulty: Intermediate

Good first issue for: External contributors who are familiar with TypeScript and want to contribute to improving the stability of the extension

[0xIchigo]

Hello @georgewrmarshall, I'd be interested in opening up a PR to implement this change! Would you be able to assign this issue to me please?

[dhruvv173]

Can I work on this? @georgewrmarshall

[0xIchigo]

Hello @georgewrmarshall - I have begun work on migrating ButtonLink to TS however I had to stop before opening up a PR as it seems that ButtonBase (#18886) is a dependency that must be updated first as, according to the documentation, "The ButtonLink is an extension of ButtonBase to support link styles".

I see that a PR is currently open to update ButtonBase (#19067) and do not mind waiting as it seems that the PR is almost complete. I was wondering if there were any other components that you need migrated first that I could work on them in the interim before opening up a PR for ButtonLink?

[georgewrmarshall]

Removing good-first-issue label until we close the Box dependency #19239