Skip to content

Commit

Permalink
Merge pull request #76 from SAP/question.isWhen.isValid
Browse files Browse the repository at this point in the history
[BUG-FIX]: Step validation - question.isValid === false && question.i…
  • Loading branch information
tomer-epstein authored Jan 2, 2020
2 parents ded0978 + a09b697 commit 4388981
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion backend/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"license": "Apache 2.0",
"description": "Provide rich user experience for Yeoman generators using VSCode extension or the browser",
"repository": "https://github.com/SAP/yeoman-ui",
"version": "0.0.17",
"version": "0.0.18",
"engines": {
"vscode": "^1.38.0"
},
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/components/Step.vue
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ export default {
deep: true,
handler() {
const invalidQuestions = _.filter(this.currentPrompt.questions, question => {
return question.isValid === false
return (question.isValid === false && question.isWhen === true)
})
this.$emit("stepvalidated", _.isEmpty(invalidQuestions))
}
Expand Down
2 changes: 1 addition & 1 deletion generator-foodq/generators/app/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ module.exports = class extends Generator {
default: "Junk food"
},
{
when: (response) => {
when: async (response) => {
this.log(response.hungry);
const promise = new Promise((resolve, reject) => {
this.log(`Purposely delaying response for 2 seconds...`);
Expand Down

0 comments on commit 4388981

Please sign in to comment.