teammates [#8450] Instructor: edit session: 'distribute points' questions: support min/max

[#8450] Instructor: edit session: 'distribute points' questions: support min/max

Open thikhinab opened this issue 2 years ago • 10 comments

Fixes #8450

Outline of Solution

Updated the the FeedbackConstantSumQuestionDetails.java with the necessary fields, validation and error checking to support min/max.
Updated the UI:
- so that the instructor can specify the min/max for distribute points among options question
- with front-end validation for min/max

Note The changes are made for constsum-options-question.

UI Question details form

Question answer form

Question additional information (Under view results):

Jul 13 '22 10:07 thikhinab

Hi @thikhinab, these parts of your pull request do not appear to follow our contributing guidelines:

Jul 13 '22 10:07 teammates-bot

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Jul 30 '22 15:07 nusoss-bot

@xyliew25 please help with review; you're working on question details during your first few weeks.

Jul 31 '22 16:07 wkurniawan07

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Aug 09 '22 15:08 nusoss-bot

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Aug 17 '22 14:08 nusoss-bot

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Aug 30 '22 15:08 nusoss-bot

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Sep 06 '22 16:09 nusoss-bot

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Sep 14 '22 14:09 nusoss-bot

@NicolasCwy @xyliew25 friendly reminder for review

Sep 16 '22 05:09 ypinhsuan

Hi @ypinhsuan sorry but I don't think I'm able to do a review anytime soon.

Sep 20 '22 04:09 xyliew25

Guys, This PR seems to be stalling (no activities for the past 7 days). :snail: :cry: Hope someone can get it to move forward again soon...

Sep 27 '22 15:09 nusoss-bot

@NicolasCwy can review this one?

Sep 27 '22 15:09 damithc

@damithc Will merge, this since we already have 2 reviews for this PR

Sep 28 '22 15:09 NicolasCwy