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

Nt/multiple choice #2877

Merged
merged 16 commits into from
May 24, 2024
Merged

Nt/multiple choice #2877

merged 16 commits into from
May 24, 2024

Conversation

noumantahir
Copy link
Collaborator

@noumantahir noumantahir commented May 22, 2024

What does this PR?

support for select multiple choices upto number mentioned in property max_choices_votes
right now, since out poll.ecency end point do not support updated schema, polls will not work as expected, for testing purposes I used poll-beta from hivehub

However, something to keep in mind, once we do update polls.ecency, our users using app variant that supports old version of polls will no longer be able to use polls as expected since updated schema returns array of choices instead of choice_num under poll_voters

UPDATE: merged poll blog icon branch into this, to show poll icon in user feed

Screenshots/Video

Screen.Recording.2024-05-22.at.18.53.20.mov
Screen.Recording.2024-05-22.at.18.41.12.mov

poll blog icon display
Screenshot 2024-05-22 at 20 27 13

@noumantahir noumantahir requested a review from feruzm May 22, 2024 14:00
@noumantahir
Copy link
Collaborator Author

added support for POLL text/tag above title along with PROMOTED text

Screenshot 2024-05-23 at 18 25 15 Screenshot 2024-05-23 at 18 25 26 Screenshot 2024-05-23 at 18 25 38

@noumantahir
Copy link
Collaborator Author

UPDATE:

refined poll choices styling a bit

  • updated border width of selected option
  • fixed text pushing count off the screen

BEFORE
Screenshot 2024-05-23 at 19 01 39

NOW
Screenshot 2024-05-23 at 18 58 40

Copy link
Member

@feruzm feruzm left a comment

Choose a reason for hiding this comment

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

Lgtm

@feruzm feruzm merged commit 9457d79 into development May 24, 2024
@feruzm feruzm deleted the nt/multiple-choice branch May 24, 2024 03:07
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