Skip to content

Commit

Permalink
Merge pull request #1309 from Amsterdam/WON-127-voeg-viewport-en-user…
Browse files Browse the repository at this point in the history
…-agent-toe-aan-feedback-message

WON-127 Add optional user-agent and screen size fields to feedback se…
  • Loading branch information
remyvdwereld authored Feb 18, 2025
2 parents 1026419 + f1c9dfe commit 2588e51
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 1 deletion.
4 changes: 4 additions & 0 deletions app/apps/feedback/serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,7 @@ class FeedbackSerializer(serializers.Serializer):
feedback = serializers.CharField()
user = serializers.HiddenField(default=serializers.CurrentUserDefault())
url = serializers.CharField()
user_agent = serializers.CharField(required=False) # Optional field for user-agent
screen = serializers.CharField(
required=False
) # Optional field for screen (viewport browser)
15 changes: 14 additions & 1 deletion app/apps/feedback/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,19 @@ def _create_slack_message_body(self, data):
{"type": "mrkdwn", "text": f"*E-mail:*\n {data['user'].email}"},
],
},
{
"type": "section",
"fields": [
{
"type": "mrkdwn",
"text": f"*User-Agent:*\n `{data['user_agent']}`",
},
{
"type": "mrkdwn",
"text": f"*Schermgrootte:*\n `{data['screen']}`",
},
],
},
{
"type": "section",
"text": {"type": "mrkdwn", "text": f"*URL:*\n <{data['url']}>"},
Expand All @@ -53,7 +66,7 @@ def _create_slack_message_body(self, data):
"type": "section",
"text": {
"type": "mrkdwn",
"text": f":loudspeaker: *Feedback: *\n ```{data['feedback']}```",
"text": f":loudspeaker: *Feedback: *\n> {data['feedback']}",
},
},
{"type": "divider"},
Expand Down

0 comments on commit 2588e51

Please sign in to comment.