Skip to content

Conversation

JolandaVerhoef
Copy link
Collaborator

There was a lot of overlap in these questions, even though the designs are very similar. I consolidated all versions and updated it according to the designs.

this CL follows up on #1024 so please merge that before this.

…mplify code

Change-Id: I006fcf3322555af4702d876130b01cc9143c58ab
…ecific data classes

Change-Id: I9f215343bcb60003eca8ef07f14413cba86469a6
Change-Id: I7a91ddbd776870cc14067d72a3376fffbff33e65
Change-Id: I89abd9b59678aabd56722572d4c2915a86c4888a
@JolandaVerhoef JolandaVerhoef requested a review from a team as a code owner November 17, 2022 16:01
Base automatically changed from jv/extract_questions to main November 21, 2022 13:48
Copy link
Collaborator

@riggaroo riggaroo left a comment

Choose a reason for hiding this comment

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

Yay, less code! Nice work 🎉

@JolandaVerhoef JolandaVerhoef merged commit d314bd2 into main Nov 21, 2022
@JolandaVerhoef JolandaVerhoef deleted the jv/improve_jetsurvey_question branch November 21, 2022 17:32
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