Skip to content

Commit

Permalink
Fix issue with onClick not working for Multiselect <Token>s
Browse files Browse the repository at this point in the history
Update readme to remember how to bump versions
  • Loading branch information
rmorse committed Mar 6, 2024
1 parent e548ad1 commit a0874d0
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
2 changes: 1 addition & 1 deletion components/multi-select-control/token.js
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ export default function Token( {
<Button
className="codeamp-components-multi-select-control__remove-token"
icon={ closeSmall }
onClick={ ! disabled && onClick }
onClick={ ! disabled ? onClick : noop }
label={ messages.remove }
aria-describedby={ `codeamp-components-multi-select-control__token-text-${ instanceId }` }
/>
Expand Down
11 changes: 11 additions & 0 deletions readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,3 +41,14 @@ This package is currently published as beta while we continue to setup the repos

All contributions are welcome, but at this early stage the priority is to setup testing and improve the documentation.



## Updating the project version number

To update the project version number, run the following command:

```
npm version <update_type>
```

Where `<update_type>` is one of the semantic versioning release types: `patch`, `minor`, or `major`. This command will update the version number in `package.json`, create a version commit, and tag the commit.

0 comments on commit a0874d0

Please sign in to comment.