fix(app): add landing-page to union of AppType#3367
Conversation
🦋 Changeset detectedLatest commit: 3397a73 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
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 |
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for a new app type called landing-page to the AppType union in the fusion-framework module. This change enables the framework to recognize and handle landing page applications in addition to the existing app types.
- Add
landing-pageas a new valid app type in the AppType union - Include appropriate changeset documentation for the patch release
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/modules/app/src/types.ts | Extends AppType union to include 'landing-page' type |
| .changeset/cold-rabbits-tickle.md | Adds changeset entry documenting the new app type addition |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
This PR modifies the same line as #3386 which has already been merged. The Current state: export type AppType = 'standalone' | 'report' | 'launcher' | 'template' | 'template-app';This PR needs to:
|
2bb883d to
747946d
Compare
|
@odinr ✅ rebase is done. Ready for your review |
747946d to
aa8fc48
Compare
aa8fc48 to
3397a73
Compare
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||
odinr
left a comment
There was a problem hiding this comment.
✅ Approved
Great work on this PR! The code changes are well-implemented and follow our coding standards.
Highlights
- ✅ Simple and correct union type addition for
landing-pageapp type - ✅ Proper changeset with appropriate patch versioning
- ✅ All validation passes (build, tests, type checking)
- ✅ Non-breaking change that cleanly extends the
AppTypeunion - ✅ Self-review checklist completed
Code Quality
The implementation is clean and follows TypeScript best practices. Adding 'landing-page' to the AppType union is straightforward and maintains type safety.
Suggestions for Future PRs
For future reference, it would be helpful to include a bit more context in the PR description, such as:
- What
landing-pageapp type represents - Why this addition is needed
- How it will be used in
fusion-portal
This helps reviewers understand the context and purpose of the change, making the review process smoother.
Ready to merge! 🚀
🔧 Admin Testing CompleteThis PR has been thoroughly tested by an admin and is ready for merge: Validation Results
Admin Actions
Status: Ready to merge This PR is being merged by admin after comprehensive testing. |
Why
Used in
fusion-portalcloses:
Check off the following:
Confirm that I checked changes to branch which I am merging into.
Confirm that the I have completed the self-review checklist.
Confirm that my changes meet our code of conduct.