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

Fix application question field #1370

Merged
merged 2 commits into from
Sep 12, 2024
Merged

Conversation

nathanclairmonte
Copy link
Contributor

@nathanclairmonte nathanclairmonte commented Sep 11, 2024

Rationale

An issue came through the TS ticketing system and was passed to us, where this application has an input field missing for the Ceph question.

Turned out that this Ceph question has a question type of multi_select, while on the frontend for canonical.com we only check for the multi select type of multi_value_multi_select, hence the problem.

Done

  • Add check for multi_select question type

QA

Issue / Card

WD-14817

Screenshots

Before fix:
image

After fix:
Screenshot 2024-09-11 193359

@webteam-app
Copy link

@nathanclairmonte nathanclairmonte merged commit 52e8e52 into main Sep 12, 2024
9 checks passed
@nathanclairmonte nathanclairmonte deleted the fix-application-question-field branch September 12, 2024 16:11
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.

3 participants