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

Validate custom components #15469

Merged
merged 3 commits into from
Jan 31, 2025
Merged

Conversation

adrinr
Copy link
Collaborator

@adrinr adrinr commented Jan 31, 2025

Description

Validate custom components. While doing this I realised we should be using the existing definitions instead us using the manifest.json, as it contains plugins and the snapshot of the definitions based on the app version.
I also renamed the feature flag, to avoid wrong usage of manifest for versions before this one.

Screenshots

Screen.Recording.2025-01-31.at.09.33.03.mov

Launchcontrol

Validate custom component settings

Copy link

linear bot commented Jan 31, 2025

Copy link

qa-wolf bot commented Jan 31, 2025

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

Copy link
Member

@aptkingston aptkingston left a comment

Choose a reason for hiding this comment

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

LGTM!

@aptkingston aptkingston merged commit 7eb9a68 into master Jan 31, 2025
20 checks passed
@aptkingston aptkingston deleted the BUDI-9010/validate-custom-components branch January 31, 2025 11:17
@github-actions github-actions bot locked and limited conversation to collaborators Jan 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants