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

Feature - Rich Text Editor #402

Merged
merged 4 commits into from
Dec 9, 2021

Conversation

calebpanza
Copy link
Contributor

@calebpanza calebpanza commented Dec 8, 2021

About

Adds a new UI element to the UI Kit: RichTextEditor which is based on Quill.js and uses a Quill-React bridge to implement

Test Instructions

Check out Storybook to see the UI element in action
Navigate to /group/{{ group title }}/email to see it in play on the actual Email Composer

Screenshots

Screen Shot 2021-12-08 at 2 29 16 PM

Closes Tickets

CFDP-1871

@vercel
Copy link

vercel bot commented Dec 8, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/christ-fellowship-church/web-app-v2/EH5hJjeayRc4ju3HhQ98UeB1EeKj
✅ Preview: https://web-app-v2-git-feature-rich-tex-adbabb-christ-fellowship-church.vercel.app

@calebpanza calebpanza changed the base branch from main to release-group-members December 8, 2021 19:38
@dzwood
Copy link
Contributor

dzwood commented Dec 9, 2021

@aSleepingPanda looks like there's some conflicts 👀

@calebpanza
Copy link
Contributor Author

@aSleepingPanda looks like there's some conflicts 👀

@dzwood resolved

Copy link
Contributor

@dzwood dzwood left a comment

Choose a reason for hiding this comment

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

I can't seem to test it from Storybook, but everything else looks good on my end 👍

@calebpanza calebpanza merged commit e462d06 into release-group-members Dec 9, 2021
@calebpanza calebpanza deleted the feature-rich-text-editor branch December 9, 2021 17:40
calebpanza added a commit that referenced this pull request Jan 4, 2022
* Feature manage group (#365)

* Update Group Single Screen (#339)

* Updates default messaging for Group Cards (#338)

* Tests written

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>

* Add new "pen" icon

* Update a handful of styles on buttons

* Linter
* Border Width now set to 2px for legibility

* Updated button style on Group Single screen

* Fixes Group Member avatar layout issues

* Removed console.log

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>
Co-authored-by: dzwood <46049974+dzwood@users.noreply.github.com>

* Edit group screen (#341)

* Update Resources and set up Group Members card

* Components for displaying a Group Member

* Stories written for all components
* Component for GroupMember
* UI element for GroupMemberStatusBadge

* Check in

* Resolves error where Modal wasn't working

* Finalize the Edit Group Member

* Add a new group member modal

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>

* Quick Updates

* Caches Group Member Search Results
* Manually update cache for Group Member _updates_
* Couple of SquareAvatar missing props

* Updated the caching logic

@see apollographql/apollo-client#6760

* New API updates

* Update icons.js

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>
Co-authored-by: dzwood <46049974+dzwood@users.noreply.github.com>

* Update Group Single Screen (#344)

* Update Group Single Screen (#339)

* Updates default messaging for Group Cards (#338)

* Tests written

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>

* Add new "pen" icon

* Update a handful of styles on buttons

* Linter
* Border Width now set to 2px for legibility

* Updated button style on Group Single screen

* Fixes Group Member avatar layout issues

* Removed console.log

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>
Co-authored-by: dzwood <46049974+dzwood@users.noreply.github.com>

* Edit group screen (#341)

* Update Resources and set up Group Members card

* Components for displaying a Group Member

* Stories written for all components
* Component for GroupMember
* UI element for GroupMemberStatusBadge

* Check in

* Resolves error where Modal wasn't working

* Finalize the Edit Group Member

* Add a new group member modal

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>

* Quick Updates

* Caches Group Member Search Results
* Manually update cache for Group Member _updates_
* Couple of SquareAvatar missing props

* Updated the caching logic

@see apollographql/apollo-client#6760

* New API updates

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>
Co-authored-by: dzwood <46049974+dzwood@users.noreply.github.com>

* Email group button (#400)

* first commit

* Button Spacing

* added route

Co-authored-by: asleepingpanda <45076058+asleepingpanda@users.noreply.github.com>

* Adds layout and route for Emailing Group (#399)

* Adds layout and route for Emailing Group

* Updated Label

* Group Member Filters (#401)

* Group Member Filters

Active, Pending, Inactive

* Mobile styling

* Feature - Rich Text Editor (#402)

* Quill React package and storybook

* Adds the Rich Text Editor to the Email Composer

* Font Size updates

* Feature email composer confirmation (#404)

* Email Confirmation dialogue

* Quick path routing clean up

* Select email recepients (#405)

* first commit

* Group Member Selection local state

* Update GroupEmailComposer.js

* Adds Email Recipients Modal

* Adds Email Recipient Modal

* commit for review

* Update UI to be inline

* Deprecated modal

* Remove console.log's

Co-authored-by: Lis Alfonso <46769629+codingwithlis@users.noreply.github.com>

* Email Recipient/Select All update

* Email a specific Group Member

* Email Confirmation Screen

* [CFDP-1896] Connected EmailComposer to API

* patch - default campus value

* Fixed styling on Group Member Detail modal

* Fixed Styling for Sender Email

Co-authored-by: Caleb Panza <caleb.panza@icloud.org>
Co-authored-by: dzwood <46049974+dzwood@users.noreply.github.com>
Co-authored-by: Lis Alfonso <46769629+codingwithlis@users.noreply.github.com>
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.

2 participants