Skip to content
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

#[132233961] Admins should be able to CRUD questions for the questionnaire #127

Open
wants to merge 42 commits into
base: master
Choose a base branch
from

Conversation

andela-bkanyolo
Copy link
Collaborator

[132233961] Admins should be able to CRUD questions for the questionnaire

What does this PR do?

This PR adds functionality that enables administrators to view, add, edit, rank and delete questions that will appear in new artisan questionnaires.

Any background context you want to provide?

New artisans to workdey have to fill out a questionnaire which will give helpful information for the vetting process. This feature enables workdey admins to modify what this questionnaire will ask for.

What are the relevant pivotal tracker stories?

132233961

Screenshots

screen shot 2016-11-28 at 15 46 02

screen shot 2016-11-28 at 15 46 59

screen shot 2016-11-28 at 15 47 06

screen shot 2016-11-28 at 15 47 17

Bernard Kanyolo added 30 commits November 10, 2016 15:34
required: false
},

{
question: "What challenge have you faced at work and how did you
deal with it?",
question: "What challenge have you faced at work and how did you deal with it?",

Choose a reason for hiding this comment

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

Line is too long. [88/80]

},

{
question: "What is your biggest weakness and how do you plan to
overcome it?",
question: "What is your biggest weakness and how do you plan to overcome it?",

Choose a reason for hiding this comment

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

Line is too long. [86/80]

@@ -0,0 +1,2 @@
module Admin::QuestionsHelper

Choose a reason for hiding this comment

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

Use nested module/class definitions instead of compact style.

@@ -0,0 +1,83 @@
class Admin::QuestionsController < ApplicationController

Choose a reason for hiding this comment

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

Use nested module/class definitions instead of compact style.

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.

3 participants