Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AppFrame - Convert to TypeScript #2098

Merged
merged 10 commits into from
May 20, 2024
Merged

Conversation

MelSumner
Copy link
Contributor

@MelSumner MelSumner commented May 15, 2024

📌 Summary

If merged, this PR converts the AppFrame component (and sub-components) to TypeScript.

🔗 External links

Jira ticket: HDS-2719


💬 Please consider using conventional comments when reviewing this PR.

@MelSumner MelSumner requested a review from a team May 15, 2024 17:28
Copy link

vercel bot commented May 15, 2024

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

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview May 20, 2024 5:51pm
hds-website ✅ Ready (Inspect) Visit Preview May 20, 2024 5:51pm

Copy link
Member

@alex-ju alex-ju 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, but it would need a rebase we've got some changes to the template-registry in the meantime

.changeset/smart-badgers-own.md Outdated Show resolved Hide resolved
Blocks: {
default: [];
};
Element: HTMLElement;
Copy link
Member

Choose a reason for hiding this comment

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

Noting here that there is no HTMLFooterElement that we could use, unfortunately.

@MelSumner MelSumner force-pushed the melsumner/hds-2719-appframe-ts branch from ba12a3e to a32efcd Compare May 20, 2024 17:47
@MelSumner MelSumner merged commit c4b79b6 into main May 20, 2024
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants