Closed gnapse closed 1 year ago
@gnapse Do you expect reviewers to test this in Reactist via Storybook (which it's what I did) or also trying to use the component in todoist-web (which I have not done)?
Do you expect reviewers to test this in Reactist via Storybook (which it's what I did) or also trying to use the component in todoist-web (which I have not done)?
Unless there's an explicit mention of this in a test plan in the PR description, I do not think it's necessary. Though it's always welcome if the reviewer thinks it's worth it to double-check.
Short description
See CHANGELOG in the diff for a summary of the changes in this PR.
PR Checklist
npm run validate
and made sure no errors / warnings were shownCHANGELOG.md
package.json
andpackage-lock.json
(npm --no-git-tag-version version <major|minor|patch>
) refnpm run build-all
)Versioning
New major release, as this introduces a breaking change (mentioned above).
Demo