Skip to content

Conversation

wobsoriano
Copy link
Member

@wobsoriano wobsoriano commented Aug 29, 2024

Description

This PR sets a default element type to the Polymorphic unstyled Astro components and addresses prop spreading to resolve TS union errors and simplify type evaluation:

Screenshot 2024-08-29 at 3 54 49 PM

With a default element:

Screenshot 2024-08-29 at 3 55 47 PM

Fixes ECO-172

Checklist

  • npm test runs as expected.
  • npm run build runs as expected.
  • (If applicable) JSDoc comments have been added or updated for any package exports
  • (If applicable) Documentation has been updated

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Copy link

changeset-bot bot commented Aug 29, 2024

🦋 Changeset detected

Latest commit: 5c54391

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@clerk/astro Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@LekoArts LekoArts enabled auto-merge (squash) August 30, 2024 08:55
Copy link
Member

@octoper octoper left a comment

Choose a reason for hiding this comment

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

🔥

@LekoArts LekoArts merged commit 3aa7798 into main Aug 30, 2024
19 checks passed
@LekoArts LekoArts deleted the rob/eco-172-typescript-error-when-using-unstyled-components branch August 30, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants