Skip to content

Conversation

@pipopotamasu
Copy link
Collaborator

@pipopotamasu pipopotamasu commented Mar 18, 2025

Added a new feature option 'entrypoint' which enables a user to choose which page they want to land on . Currently, they can choose 'labs' or 'catalog' for this option which navigates them to Labs page or Catalog page. This option is available when they use 'platform' feature.

…og page for platform feature

Added a new feature option 'entrypoint' which enables a user to choose which page they want to land
on . Currently, they can choose 'labs' or 'catalog' for this option which navigates them to Labs
page or Catalog page. This option is available when they use 'platform' feature.
@pipopotamasu pipopotamasu changed the title feat(feature option): new feature option to navigate to Labs or Catal… feat(feature option): new feature option to navigate to Labs or Catalog page Mar 18, 2025
Copy link

@brandonbay brandonbay 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 to me!

featureType: FeatureType,
featureOptions: FeatureOptions<FeatureType>
) => {
if ("entrypoint" in featureOptions) {

Choose a reason for hiding this comment

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

Is this if necessary to confirm the type to tsc?

Copy link
Collaborator Author

@pipopotamasu pipopotamasu Mar 18, 2025

Choose a reason for hiding this comment

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

This is not for tsc.

We would like to remove entrypoint point option when it is given. That's why I put this if condition. Technically, we can merge this if with the following one but I wanted to make it separated for readability

@fallenAfter
Copy link

Looks good to me!

@pipopotamasu pipopotamasu merged commit 91c3ec9 into master Mar 31, 2025
3 checks passed
@pipopotamasu pipopotamasu deleted the entrypoint branch March 31, 2025 13:58
@fullscript-devops
Copy link
Collaborator

🎉 This PR is included in version 2.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants