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

feat: add boolean and number transformers #2333

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dafnik
Copy link

@Dafnik Dafnik commented Jan 3, 2024

adding to ng-dropdown-panel, ng-option and ng-select

@pavankjadda
Copy link
Collaborator

pavankjadda commented Jan 10, 2024

@Dafnik what does this achieve?

@pavankjadda pavankjadda self-requested a review January 10, 2024 15:13
@pavankjadda pavankjadda self-assigned this Jan 10, 2024
@pavankjadda pavankjadda added the needs info Needs more information label Jan 10, 2024
@Dafnik
Copy link
Author

Dafnik commented Jan 11, 2024

Ohh, sorry for not explaining earlier.

It allows boolean and number inputs to passed like a string.

<ng-select-example multiple="true" />
// Instead of
<ng-select-example [multiple]="true" />

or even just

<ng-select-example multiple />

to set a boolean to true.

The docs are really none-existent on this.
https://angular.io/api/core/booleanAttribute
https://angular.io/api/core/numberAttribute

Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions 🎆

@github-actions github-actions bot added the stale label Mar 12, 2024
@github-actions github-actions bot closed this Mar 19, 2024
@pavankjadda pavankjadda reopened this Mar 19, 2024
@github-actions github-actions bot removed the stale label Mar 20, 2024
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions 🎆

@github-actions github-actions bot added the stale label May 19, 2024
Copy link
Collaborator

@pavankjadda pavankjadda left a comment

Choose a reason for hiding this comment

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

can you rebase with master and fix conflicts?

@pavankjadda
Copy link
Collaborator

@Dafnik I like the PR. Fix conflicts and push new update

@pavankjadda pavankjadda added new feature and removed stale needs info Needs more information labels May 23, 2024
adding to ng-dropdown-panel, ng-option and ng-select
@Dafnik Dafnik force-pushed the feature/add-input-transformers branch from 87d4ac7 to 0b4ac58 Compare May 24, 2024 12:53
@Dafnik
Copy link
Author

Dafnik commented May 24, 2024

Done!

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

Successfully merging this pull request may close these issues.

None yet

2 participants