Skip to content

add feedback form to answer detail page#3

Open
dave2205 wants to merge 1 commit intomainfrom
AH-3
Open

add feedback form to answer detail page#3
dave2205 wants to merge 1 commit intomainfrom
AH-3

Conversation

@dave2205
Copy link
Copy Markdown
Collaborator

No description provided.

@dave2205 dave2205 requested a review from capsert April 30, 2024 09:25
Comment thread apps/feedback/models/models.py Outdated
help_text=_("(0 = not clear, 5 = very clear)"), null=True, blank=True
)
reliability_grade = models.PositiveSmallIntegerField(
verbose_name=_("Do you find this KlimaatHelpdesk article reliable?"),
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hier staat nog KlimaatHelpdesk

Comment thread apps/feedback/models/models.py Outdated
user_email = models.EmailField(
verbose_name=_("Email"),
help_text=_("We will forward the expert's response when we receive it."),
null=True,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

null=True hoeft nooit bij textvelden met blank. Blank zorgt er voor dat het db veld gevuld wordt met een lege string ("")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Dit geldt voor alle text/char velden in dit model.

Comment thread apps/feedback/forms.py Outdated
required=True,
)
reliability_grade = forms.IntegerField(
label=_("Do you find this KlimaatHelpdesk article reliable?*"),
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hier staat nog KlimaatHelpdesk

Comment thread apps/feedback/models/pages.py Outdated

template = "feedback/feedback_form_page.html"
context = self.get_context(request)
Answer = apps.get_model(app_label='cms', model_name='Answer')
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Waarom haal je op deze manier het model op? ipv gewoon importeren?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

omdat ik wagtail_helpdesk nog via pip install -e had geinstalleerd en dan werkt importeren niet. ik pas het aan.

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