Skip to content

Conversation

@john-traas
Copy link
Contributor

@john-traas john-traas commented Nov 8, 2024

Fixes https://github.com/Lundalogik/crm-feature/issues/4455

May need separate PR's in both building blocks and lime-crm-components to then import and use the Type definition.

@github-actions
Copy link

github-actions bot commented Nov 8, 2024

Documentation has been published to https://lundalogik.github.io/lime-elements/versions/PR-3293/

node: CustomElementDefinition[];
};

export type EditorUiTypes = 'standard' | 'minimal';
Copy link
Contributor

Choose a reason for hiding this comment

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

Singular ☺️

Suggested change
export type EditorUiTypes = 'standard' | 'minimal';
export type EditorUiType = 'standard' | 'minimal';

Copy link
Contributor Author

@john-traas john-traas Nov 11, 2024

Choose a reason for hiding this comment

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

😄 Updated

@john-traas john-traas marked this pull request as ready for review November 11, 2024 15:00
@john-traas john-traas force-pushed the add-ui-types-as-a-type branch 2 times, most recently from e18c3f8 to c65260e Compare November 11, 2024 15:09
@john-traas john-traas requested a review from a team as a code owner November 11, 2024 15:36
node: CustomElementDefinition[];
};

export type EditorUiType = 'standard' | 'minimal';
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a missing release tag on this type

@john-traas john-traas force-pushed the add-ui-types-as-a-type branch from 71ac81e to 6b070d1 Compare November 12, 2024 08:00
@john-traas john-traas merged commit 56f3a24 into main Nov 14, 2024
10 checks passed
@john-traas john-traas deleted the add-ui-types-as-a-type branch November 14, 2024 12:48
@lime-opensource
Copy link
Collaborator

🎉 This PR is included in version 37.66.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