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 help overlay and add feature flag #1528

Merged

Conversation

alexanderleegs
Copy link
Contributor

This PR adds a help overlay for texts cards. It also adds a feature flag into the text cards component.
Screenshot 2023-09-27 at 10 24 15 AM

@alexanderleegs alexanderleegs requested a review from a team September 27, 2023 02:24
@mergify
Copy link

mergify bot commented Sep 27, 2023

⚠️ The sha of the head commit of this PR conflicts with #1505. Mergify cannot evaluate rules on this PR. ⚠️

@alexanderleegs alexanderleegs temporarily deployed to staging September 27, 2023 02:34 — with GitHub Actions Inactive
@alexanderleegs alexanderleegs temporarily deployed to staging September 27, 2023 02:36 — with GitHub Actions Inactive
@alexanderleegs alexanderleegs temporarily deployed to staging September 27, 2023 02:38 — with GitHub Actions Inactive
@alexanderleegs alexanderleegs temporarily deployed to staging September 27, 2023 07:15 — with GitHub Actions Inactive
@alexanderleegs alexanderleegs merged commit a723c2d into feat/text-cards-panel Sep 27, 2023
@mergify mergify bot deleted the feat/is-457-text-cards-help-overlay branch September 27, 2023 09:00
alexanderleegs added a commit that referenced this pull request Sep 27, 2023
* feat: add help overlay and add feature flag

* chore: rearrange resource position in add section
alexanderleegs added a commit that referenced this pull request Sep 27, 2023
* chore: add validators

* chore: add constants

* chore: add new textcard component types

* chore: add placeholder preview section

* chore: add new homepageDroppableZone type

* Feat: add textCards side panel

* feat: update useDrag to work with subtextcards

* feat: add text card parsing and change handler

* Fix: add button styles and restrict cards to 4

* fix: disable save on card error

* fix: rebase errors

* fix: remove comments

* fix: cast text cards once

* chore: remove unused updateElement

* fix: bug when attempting to create 2 textcard sections before adding text cards

Lodash behaviour here is buggy - _.set sets the fields in multiple array items for section data, which was causing errors

* chore: rename textcardcard

* feat: add help overlay and add feature flag (#1528)

* feat: add help overlay and add feature flag

* chore: rearrange resource position in add section

* IS-456: Text Cards Previews (#1516)

* wip: basic cms preview

* feat: refactor styles

Co-authored-by: Alexander Lee <[email protected]>

* feat: enable previews on text-cards

* fix: lint issues

* feat: update previews to match template

Co-authored-by: Alexander Lee <[email protected]>

---------

Co-authored-by: Alexander Lee <[email protected]>

* chore: refactor homepage to if/else to preserve existing behaviour (#1527)

* chore: refactor homepage to if/else to preserve existing behaviour

* fix: rebase errors

* fix: check for empty string

* fix: rename var

* fix: add type guards to create and delete and add comments

* chore: change to unknown

---------

Co-authored-by: Harish <[email protected]>
Co-authored-by: Alexander Lee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants