Skip to content

fix: type link variants#67

Merged
lihbr merged 3 commits intomainfrom
lh/link-variant-generic
Jan 20, 2025
Merged

fix: type link variants#67
lihbr merged 3 commits intomainfrom
lh/link-variant-generic

Conversation

@lihbr
Copy link
Member

@lihbr lihbr commented Jan 16, 2025

Resolves:

Description

Add missing types for link variants.

Checklist

  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

image

How to QA 1

  • Checkout branch, install, and build
  • npm link
  • In a Slice Machine project: npm update @prismicio/client && npm link prismic-ts-codegen
  • Add a link field with variants
  • Check types

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

Copy link
Contributor

@xrutayisire xrutayisire left a comment

Choose a reason for hiding this comment

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

So nice to have this! Nice work 🙂

@lihbr lihbr merged commit 2f4ecd3 into main Jan 20, 2025
10 checks passed
@lihbr lihbr deleted the lh/link-variant-generic branch January 20, 2025 13:48
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.

3 participants