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

Survey driven discussion topics #703

Merged
merged 14 commits into from Nov 28, 2013

Conversation

Projects
None yet
3 participants
@benfoxall
Contributor

benfoxall commented Nov 26, 2013

This allows the juvia discussion topic to be set in the surveyor config.

The :discussion_topic needs to be added to each question - it has been added to a development survey in 6d92d9b

I've had a look on juvia, there is one discussion that could become inaccessible: https://certificates.theodi.org/comment?back=%2Fsurveys%2Fgb%2F9FNXesLB6Q%2Ftake&key=contactUrl&title=Where+can+people+find+out+how+to+contact+someone+with+questions+about+this+data%3F&topic=%2Fsurveys%2Fquestions%2FcontactUrl

@benfoxall

This comment has been minimized.

Show comment
Hide comment
@benfoxall

benfoxall Nov 27, 2013

Contributor

From the readme:

:discussion_topic - Sets the topic (and enables) juvia discussion for this question (accessible by clicking "Ask for help with this question"). The :help_text value for this question will appear at the top of the discussion page.

Contributor

benfoxall commented Nov 27, 2013

From the readme:

:discussion_topic - Sets the topic (and enables) juvia discussion for this question (accessible by clicking "Ask for help with this question"). The :help_text value for this question will appear at the top of the discussion page.

@benfoxall

This comment has been minimized.

Show comment
Hide comment
@benfoxall

benfoxall Nov 27, 2013

Contributor
  • tie help text to particular question. More than one question/jurisdiction may share the same question topic, would be better to pass the question_id to the comment page in case the help text has been tweaked for a particular jurisdiction.
Contributor

benfoxall commented Nov 27, 2013

  • tie help text to particular question. More than one question/jurisdiction may share the same question topic, would be better to pass the question_id to the comment page in case the help text has been tweaked for a particular jurisdiction.

peterjwest added a commit that referenced this pull request Nov 28, 2013

@peterjwest peterjwest merged commit 7b26088 into master Nov 28, 2013

1 check passed

default Build #5 succeeded in 7 min 10 sec
Details

@peterjwest peterjwest deleted the survey-driven-topics branch Nov 28, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment