-
Notifications
You must be signed in to change notification settings - Fork 9
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
release(v0.45.0): merge to master #1520
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* fix(chromatic): only do path checking on push * fix(chromatic): also build on deps changes
Release/0.44.0
* feat(template): add ffs as a manual check-in * feat(template): link do doc for sot
* chore(editable): promote to shared component * feat: introduce new help overlay for add section button * feat: add storybook for add section button * chore: remove unneeded story parameter * chore: update images for text cards and announcements * fix: adjust story to be more extendible
* feat(announcements): add SCSS styles from template * feat(announcements): introduce preview component * chore: sync padding and margin helpers * chore: sync template changes to CMS preview * feat(announcements): define shape * feat(validators): announcement validators * feat(announcement): add announcement block * feat(edit homepage): add in announcement block * feat(useDrag): update draggable * fix(announcement): interface bugs * fix(announcement): bugs after interface change * feat(storybook): attempt to get out storybook * chore(storybook): remove storybook" The purpose of the commit was to have history of previous attempt vs commenting it in production code * fix(announcementbody): dont trigger re-render * fix(announcement): fix bugs wrt shape normalise datetime * chore(edit homepage): rm console log * fi(announcement): on drag bug fix * chore(edit homepage): rm console.log * fix(validations): should throw error for empty str * chore(announcement body): dynamically retrieve max announcement * chore(homepage): remove redundant check * chore(edithomepage): remove redundant console.log * chore(edit homepage): modify this to announcements * chore(announcement body): rm redundant placeholder * chore(edithomepage): rm redundant array * IS-516 link to jira * chore(useDrag): change verb * chore(edit homepage): clarity in comment * fix(announcement): use func to get new datetime * refactor(validator): change naming * refactor(announcement): mutate at data level * fix(announcement body): copy changes * fix(announcement): copy changes for clarity * fix(announcement date): show invalid dates * style(copy): change copy * fix(announcement): copy changes * fix(announcement): have 1 announcemnt by default * build(de): fix failing build * fix(homapage prevew): rebase errors * chore(validators): rm console logs * feat(feature flag): add partial ff for announment * chore(announcemnt body): rm console log * chore(announcementBody): comments for clarity * fix(typing): relax constraint * chore(announcementbody): rm redundant console.log * refactor(editHomepage): rm redundant spread * style(edit homepage): add comments for clarity * fix(announcements): empty errors bug * chore(edit homepage): rm redundant func call * chore(edit homepage): clarify comment * fix(imports): fix imports after rebase * fix(imports): issues after rebase --------- Co-authored-by: Hsu Zhong Jun <[email protected]>
|
dcshzj
approved these changes
Sep 25, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What's Changed
Full Changelog: v0.44.0...v0.45.0