Enum - Multi Selection for vue vanilla by glengemann · Pull Request #2261 · eclipsesource/jsonforms

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@glengemann

Copy link

Contributor

Hi!

I want to add a new control to support multiple checkboxes.

@netlify

Copy link

netlify bot commented

Jan 25, 2024

edited

Loading

Deploy Preview for jsonforms-examples ready!

Name Link
🔨 Latest commit bb5f2b4
🔍 Latest deploy log https://app.netlify.com/sites/jsonforms-examples/deploys/65b28e27c982a6000825ad23
😎 Deploy Preview https://deploy-preview-2261--jsonforms-examples.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.


To edit notification comments on pull requests, go to your Netlify site configuration.

@CLAassistant

Copy link

CLAassistant commented

Jan 25, 2024

edited

Loading

CLA assistant check
All committers have signed the CLA.

@lucas-koehler

Copy link

Contributor

Hi @glengemann , thanks for your PR ❤️ We'll try to have a look soon.

@coveralls

Copy link

Coverage Status

coverage: 84.796%. remained the same
when pulling bb5f2b4 on glengemann:vue-enum-multi-selection-control
into 9d3f706 on eclipsesource:master.

Copy link

Contributor

@lucas-koehler lucas-koehler left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the contribution :)

@lucas-koehler lucas-koehler merged commit b80191e into eclipsesource:master

Feb 2, 2024

@lucas-koehler lucas-koehler added this to the 3.3 milestone

Feb 2, 2024

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

Reviewers

@lucas-koehler lucas-koehler lucas-koehler approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

3.3

Development

Successfully merging this pull request may close these issues.

4 participants

@glengemann @CLAassistant @lucas-koehler @coveralls