Skip to content

Conversation

@adboio
Copy link

@adboio adboio commented Jan 22, 2026

💡 Motivation and Context

need to support thumbs up/down to match other SDKs

💚 How did you test it?

Screenshot 2026-01-22 at 2.56.39 PM.png

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

Copy link
Author

adboio commented Jan 22, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@adboio adboio changed the base branch from kyle/improve-survey-color-consistency to graphite-base/257 January 22, 2026 23:04
@adboio adboio force-pushed the graphite-base/257 branch from d529a78 to c739d2f Compare January 22, 2026 23:05
@adboio adboio force-pushed the 01-22-chore_surveys_enable_thumb_survey_display branch from 2dfd4c6 to 3a513f6 Compare January 22, 2026 23:05
@adboio adboio changed the base branch from graphite-base/257 to kyle/improve-survey-color-consistency January 22, 2026 23:05
@adboio adboio requested a review from a team January 22, 2026 23:19
@adboio adboio marked this pull request as ready for review January 22, 2026 23:19
@adboio adboio force-pushed the kyle/improve-survey-color-consistency branch from c739d2f to 1f5f2cf Compare January 22, 2026 23:30
@adboio adboio force-pushed the 01-22-chore_surveys_enable_thumb_survey_display branch from 3a513f6 to 0252499 Compare January 22, 2026 23:30
Copy link
Collaborator

@ioannisj ioannisj left a comment

Choose a reason for hiding this comment

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

LG. Something that just occurred to me, is the order of the thumbs up/down icons (LTR) intentional? (I see iOS PR does the same, so probably yes) I'm pretty sure I've seen thumbs up on the right in other survey tools, and it also goes against the order of our emoji rating.

Copy link
Collaborator

@ioannisj ioannisj left a comment

Choose a reason for hiding this comment

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

Just saw there are some build error and CI is not happy, let's address these first

@adboio adboio force-pushed the 01-22-chore_surveys_enable_thumb_survey_display branch from 0252499 to 64933a8 Compare January 23, 2026 18:01
@adboio adboio force-pushed the kyle/improve-survey-color-consistency branch from 1f5f2cf to ae18fcb Compare January 23, 2026 18:01
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.

3 participants