Problem/Motivation

The question is not required.

At the moment one can configure a faq field to be required but only the answer is validated to be filled in.

Proposed resolution

Make the question required as well.

Remaining tasks

Add a constraint on the question field.

User interface changes

none

API changes

none

Data model changes

Add 'not null' on the question column.

CommentFileSizeAuthor
#5 2976550_5.patch3.09 KBmpp
#4 2976550_4.patch2.43 KBmpp
#4 interdiff_4.txt1.24 KBmpp
#2 both_question_and_answer_should_be_required-2976550-2.patch2.4 KBsergiu stici

Comments

mpp created an issue. See original summary.

sergiu stici’s picture

sergiu stici commented
StatusFileSize
new both_question_and_answer_should_be_required-2976550-2.patch2.4 KB

I created the patch, you can change this options in "Manage form display".

sergiu stici’s picture

sergiu stici commented
Status: Active » Needs review
mpp’s picture

mpp commented
StatusFileSize
new interdiff_4.txt1.24 KB
new 2976550_4.patch2.43 KB

Thanks @Sergiu Stici, I added $this->t() for the translations.

mpp’s picture

mpp commented
StatusFileSize
new 2976550_5.patch3.09 KB

Also added the settings to the schema.

pifagor’s picture

pifagor commented
Status: Needs review » Reviewed & tested by the community

  • pifagor committed 51c430e on 8.x-1.x authored by mpp 
    Issue #2976550 by mpp, Sergiu Stici, pifagor: Both question and answer...
pifagor’s picture

pifagor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.