-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
wip: global polls #86
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
* chore: design first iteration of refined polls * refactor: break code down into components * refactor: further break code down into components * chore: dummy implementation * refactor: use descriptive class names * chore: minor styling changes * chore: tinker with typescript * feat: implement prototype of how results are shown * chore: minor style changes * refactor: break down logic * chore: add dummy content for demo purposes * chore: minor styling changes * chore: add boilerplate state and model * chore: work in progress * Squashed commit of the following: commit 69b47f9 Author: IanM <[email protected]> Date: Thu Feb 1 09:25:46 2024 +0000 add isGlobal peroperty to js model commit d130f6b Author: IanM <[email protected]> Date: Thu Feb 1 09:21:04 2024 +0000 list polls only shows global polls commit 0a4dcf4 Author: IanM <[email protected]> Date: Wed Jan 31 16:49:15 2024 +0000 add unauthorized api test commit 6e52663 Author: IanM <[email protected]> Date: Wed Jan 31 16:45:33 2024 +0000 fix: phpstan error commit a322750 Author: StyleCI Bot <[email protected]> Date: Wed Jan 31 16:42:20 2024 +0000 Apply fixes from StyleCI commit c8e60a0 Author: IanM <[email protected]> Date: Wed Jan 31 16:42:10 2024 +0000 feat: allow creation of polls without post id (global poll) commit 7abeebe Author: IanM <[email protected]> Date: Wed Jan 31 11:49:25 2024 +0000 ensure poll is associated with postId commit 20b5917 Author: StyleCI Bot <[email protected]> Date: Wed Jan 31 11:45:37 2024 +0000 Apply fixes from StyleCI commit fa69a3c Author: IanM <[email protected]> Date: Wed Jan 31 11:45:24 2024 +0000 test: create post poll via api commit 97d6348 Author: IanM <[email protected]> Date: Wed Jan 31 08:34:08 2024 +0000 fix: provider visibility commit 03d355f Author: IanM <[email protected]> Date: Wed Jan 31 08:32:26 2024 +0000 fix: provider visibility commit 325cc32 Author: StyleCI Bot <[email protected]> Date: Wed Jan 31 08:30:15 2024 +0000 Apply fixes from StyleCI commit 822ddbf Author: IanM <[email protected]> Date: Wed Jan 31 08:29:56 2024 +0000 chore: add test for existing create poll functionality via posts commit 4799bba Author: StyleCI Bot <[email protected]> Date: Tue Jan 30 14:37:24 2024 +0000 Apply fixes from StyleCI commit 4d197f3 Author: IanM <[email protected]> Date: Tue Jan 30 14:37:13 2024 +0000 use sort commit f897796 Author: IanM <[email protected]> Date: Tue Jan 30 14:26:03 2024 +0000 chore: ts-ignore commit f1e00db Author: IanM <[email protected]> Date: Tue Jan 30 14:22:36 2024 +0000 fix: return type commit d56bf39 Author: StyleCI Bot <[email protected]> Date: Tue Jan 30 14:18:49 2024 +0000 Apply fixes from StyleCI commit d49c1f9 Author: IanM <[email protected]> Date: Tue Jan 30 14:18:37 2024 +0000 feat: list polls controller commit 8fb8cc6 Author: IanM <[email protected]> Date: Tue Jan 30 14:18:13 2024 +0000 chore: format commit ee37ee6 Author: IanM <[email protected]> Date: Tue Jan 30 14:15:49 2024 +0000 fix: some model props incorrect commit 1857a44 Author: StyleCI Bot <[email protected]> Date: Tue Jan 30 08:16:56 2024 +0000 Apply fixes from StyleCI commit 4b461a5 Author: IanM <[email protected]> Date: Tue Jan 30 08:16:27 2024 +0000 wip: add polls route, begin to setup state, controls, etc commit adf5393 Author: IanM <[email protected]> Date: Mon Jan 29 16:22:08 2024 +0000 add nav item commit d4032dc Author: IanM <[email protected]> Date: Mon Jan 29 16:09:52 2024 +0000 chore: ts commit 7365c67 Author: IanM <[email protected]> Date: Mon Jan 29 14:01:09 2024 +0000 chore: add tests for forum serializer props (#85) * chore: forum serializer test * Apply fixes from StyleCI --------- Co-authored-by: StyleCI Bot <[email protected]> commit f460073 Author: IanM <[email protected]> Date: Mon Jan 29 11:46:12 2024 +0000 chore: cleanup extend, create serializer mutators (#84) * chore: cleanup extend, create serializer mutators * Apply fixes from StyleCI --------- Co-authored-by: StyleCI Bot <[email protected]> commit 9acbd74 Author: IanM <[email protected]> Date: Mon Jan 29 11:10:32 2024 +0000 Chore: include discuss link commit a2af115 Author: IanM <[email protected]> Date: Mon Jan 29 11:00:51 2024 +0000 chore: use 1.x workflows * create PollForm and adapt Create and EditPollModal * Apply fixes from StyleCI * wip * style: format code * feat: add start global poll button * feat: add placeholder string keys * chore: troubleshooting * - remove legacy code from directory - add simple acl abstraction - add compose polls page * - add compose poll hero - adapt CreatePollHandler to the new data structure (similar to edit) - add translations * - pseudo fix poll visibility scope, requires review * - implement frontend list and edit view for poll * Apply fixes from StyleCI * - implement poll detail view * - format code * - update poll detail view - enhance poll list view * refactor: change class name * refactor: move edit, show results and delete buttons into controls dropdown. * refactor: remove wrong css classes and add necessary ones * feat: add missing en text keys * feat: add link to polls page in session dropdown * chore: cleanup * Squashed commit of the following: commit ce20ae8 Author: StyleCI Bot <[email protected]> Date: Thu Feb 15 12:19:33 2024 +0000 Apply fixes from StyleCI commit f2e6f32 Author: IanM <[email protected]> Date: Thu Feb 15 12:19:14 2024 +0000 feat: optional poll subtitle commit 69b47f9 Author: IanM <[email protected]> Date: Thu Feb 1 09:25:46 2024 +0000 add isGlobal peroperty to js model commit d130f6b Author: IanM <[email protected]> Date: Thu Feb 1 09:21:04 2024 +0000 list polls only shows global polls commit 0a4dcf4 Author: IanM <[email protected]> Date: Wed Jan 31 16:49:15 2024 +0000 add unauthorized api test commit 6e52663 Author: IanM <[email protected]> Date: Wed Jan 31 16:45:33 2024 +0000 fix: phpstan error commit a322750 Author: StyleCI Bot <[email protected]> Date: Wed Jan 31 16:42:20 2024 +0000 Apply fixes from StyleCI commit c8e60a0 Author: IanM <[email protected]> Date: Wed Jan 31 16:42:10 2024 +0000 feat: allow creation of polls without post id (global poll) commit 7abeebe Author: IanM <[email protected]> Date: Wed Jan 31 11:49:25 2024 +0000 ensure poll is associated with postId commit 20b5917 Author: StyleCI Bot <[email protected]> Date: Wed Jan 31 11:45:37 2024 +0000 Apply fixes from StyleCI commit fa69a3c Author: IanM <[email protected]> Date: Wed Jan 31 11:45:24 2024 +0000 test: create post poll via api commit 97d6348 Author: IanM <[email protected]> Date: Wed Jan 31 08:34:08 2024 +0000 fix: provider visibility commit 03d355f Author: IanM <[email protected]> Date: Wed Jan 31 08:32:26 2024 +0000 fix: provider visibility commit 325cc32 Author: StyleCI Bot <[email protected]> Date: Wed Jan 31 08:30:15 2024 +0000 Apply fixes from StyleCI commit 822ddbf Author: IanM <[email protected]> Date: Wed Jan 31 08:29:56 2024 +0000 chore: add test for existing create poll functionality via posts commit 4799bba Author: StyleCI Bot <[email protected]> Date: Tue Jan 30 14:37:24 2024 +0000 Apply fixes from StyleCI commit 4d197f3 Author: IanM <[email protected]> Date: Tue Jan 30 14:37:13 2024 +0000 use sort commit f897796 Author: IanM <[email protected]> Date: Tue Jan 30 14:26:03 2024 +0000 chore: ts-ignore commit f1e00db Author: IanM <[email protected]> Date: Tue Jan 30 14:22:36 2024 +0000 fix: return type commit d56bf39 Author: StyleCI Bot <[email protected]> Date: Tue Jan 30 14:18:49 2024 +0000 Apply fixes from StyleCI commit d49c1f9 Author: IanM <[email protected]> Date: Tue Jan 30 14:18:37 2024 +0000 feat: list polls controller commit 8fb8cc6 Author: IanM <[email protected]> Date: Tue Jan 30 14:18:13 2024 +0000 chore: format commit ee37ee6 Author: IanM <[email protected]> Date: Tue Jan 30 14:15:49 2024 +0000 fix: some model props incorrect commit 1857a44 Author: StyleCI Bot <[email protected]> Date: Tue Jan 30 08:16:56 2024 +0000 Apply fixes from StyleCI commit 4b461a5 Author: IanM <[email protected]> Date: Tue Jan 30 08:16:27 2024 +0000 wip: add polls route, begin to setup state, controls, etc commit adf5393 Author: IanM <[email protected]> Date: Mon Jan 29 16:22:08 2024 +0000 add nav item commit d4032dc Author: IanM <[email protected]> Date: Mon Jan 29 16:09:52 2024 +0000 chore: ts commit 7365c67 Author: IanM <[email protected]> Date: Mon Jan 29 14:01:09 2024 +0000 chore: add tests for forum serializer props (#85) * chore: forum serializer test * Apply fixes from StyleCI --------- Co-authored-by: StyleCI Bot <[email protected]> commit f460073 Author: IanM <[email protected]> Date: Mon Jan 29 11:46:12 2024 +0000 chore: cleanup extend, create serializer mutators (#84) * chore: cleanup extend, create serializer mutators * Apply fixes from StyleCI --------- Co-authored-by: StyleCI Bot <[email protected]> commit 9acbd74 Author: IanM <[email protected]> Date: Mon Jan 29 11:10:32 2024 +0000 Chore: include discuss link commit a2af115 Author: IanM <[email protected]> Date: Mon Jan 29 11:00:51 2024 +0000 chore: use 1.x workflows * chore: cleanup * Apply fixes from StyleCI * Refactor Poll related components and methods for better readability and error handling * refactor: Add subtitle field to PollForm component * Add check for undefined voteCount and update localization strings in PollListItem component * Refactor PollResults component to use real data and add overallVoteCount method to PollState * bugfix: fix bug about saving polls from posts etc. and add missing texts * fix tests * minor cleanup * switch to yarn * more cleanup * fix compose route reload * fix create poll * fix edit existing post poll * create dedicated poll view route * refactor: add PollViewPage * refactor: merged Subtitle and title component into PollView using ListItems & the vote button is now showing up and working * format * fix poll compose * update icons, begin removing acl * fix edit polls * refactor: use ItemList approach for form part in PollView * bugfix: fix when which data is provided in the form for the options for create and edit * remove acl * export components * export states * add nav to poll compose * refactor: improve markup and css classes assignment for PostListItem * style: improve Poll list style * build js for demo/alpha release * format and build * style: add basic style for poll result bar and center icon in poll composer icon buttons * refactor: add class and text key to poll modal delete button * feat: add percentage position indicator in poll result bar * format * feat: use submit button as separate component for possible customization (animation, etc) via extend function * Build * fix: change syntax for delete success message * feat: add sr only text to form * feat: improve accessibility * chore: remove comments * feat: further improve accessibility * chore: missing code replication Perhaps it would make sense to refactor this.. * feat: use poll question as meta title * refactor: use other icons for poll controls Use the same as when editing or deleting discussions. * chor: move PollForm to tsx format * chor: move both Modal to tsx format * wip: pollshowcase + refactoring --------- Co-authored-by: Davide Iadeluca <[email protected]> Co-authored-by: StyleCI Bot <[email protected]> Co-authored-by: Gianni Guida <[email protected]> Co-authored-by: IanM <[email protected]>
BREAKING CHANGE: Change markup to improve SEO
imorland
commented
Apr 9, 2024
* style: run formatter * chore: remove empty rule * chore: remove some unused css rules * chore: remove commented out css rules * feat!: first attempt to improve accessibility on PollOption BREAKING CHANGE: The changed markup might have unintended side effects related to styling for third party vendors. * fix: changes in selected options where not displayed * chore: remove `aria-describedby` on input * fix: add missing id to Poll text answer * chor: update dist files because of merge --------- Co-authored-by: Simon Hiller <[email protected]>
013f81a
to
6824eef
Compare
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.
Admin setting to enable/disable global polls (disabled by default)
![image](https://private-user-images.githubusercontent.com/16573496/317529077-ca08da4b-ca67-43d6-a780-b6fc831200e0.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYwNjM1NjQsIm5iZiI6MTcyNjA2MzI2NCwicGF0aCI6Ii8xNjU3MzQ5Ni8zMTc1MjkwNzctY2EwOGRhNGItY2E2Ny00M2Q2LWE3ODAtYjZmYzgzMTIwMGUwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTExVDE0MDEwNFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWU5N2Q4YzViYTY5Nzg1MDFkYzA4Mzg3ZGM2NDhiZTJkMjYwODcxZjc3ODBlM2E1MzhiMTU3Zjg1NTQ2Y2ExZDYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.jsUusUy-oD4mLhMsFVNpkn3aTjlfEqyM7-JzwM1SF7s)
Polls showcase page (for global polls)
![image](https://private-user-images.githubusercontent.com/16573496/310928231-ec4d3506-dfde-4efd-923e-b013635ad1d5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYwNjM1NjQsIm5iZiI6MTcyNjA2MzI2NCwicGF0aCI6Ii8xNjU3MzQ5Ni8zMTA5MjgyMzEtZWM0ZDM1MDYtZGZkZS00ZWZkLTkyM2UtYjAxMzYzNWFkMWQ1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTExVDE0MDEwNFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTZlMjllMjI2Mzk4YjE3ZmEwNzI5NjdkMWJlMGMwOTAzYTBlODBlMzcwZGM5YzBiNzY0NTQwNzkzMmU3OTY1YjEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.HERBK9RRrXRApkd0b6f0xzwl-U5sY6Ot7H1P1HIBUHg)
Polls now support an image to be uploaded, as well as a subtitle - both are optional
![image](https://private-user-images.githubusercontent.com/16573496/310928493-cdaaf30f-d5e5-4ce6-9b4a-fe38d4f292e5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjYwNjM1NjQsIm5iZiI6MTcyNjA2MzI2NCwicGF0aCI6Ii8xNjU3MzQ5Ni8zMTA5Mjg0OTMtY2RhYWYzMGYtZDVlNS00Y2U2LTliNGEtZmUzOGQ0ZjI5MmU1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTExVDE0MDEwNFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTFhNjE3NTU4MDRjYWQ1NjgzM2MyNGI2YTBhOGMxODMxMmE0MzNkZTViY2E4OWI0ZGI3ZWQzOWEyMGI4ZTM0ZjImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.WLUelWM6Dvi-3iGfIgtxqRfiQ_RD8UaQho4nPyrzL-s)