Skip to content

Requests List showing completed Processes#605

Merged
velkymx merged 12 commits intodevelopfrom
feature/582
Aug 28, 2018
Merged

Requests List showing completed Processes#605
velkymx merged 12 commits intodevelopfrom
feature/582

Conversation

@danloa
Copy link
Contributor

@danloa danloa commented Aug 24, 2018

  • Added assert in test to validate that the requests that are open (status TO_DO) are show in the requests/list
  • Added condition in RequestsController to show just open conditions.
  • Name of file preferences.vue changed to Preferences.vue (first letter in uppercase). The lowercase name was generating an error with npm run dev
    Ref: #582

@danloa danloa requested a review from velkymx August 24, 2018 20:55
Copy link
Contributor

@velkymx velkymx left a comment

Choose a reason for hiding this comment

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

I'm seeing the same processes on both the in progress and completed lists.

@velkymx velkymx merged commit ba3f43c into develop Aug 28, 2018
@velkymx velkymx deleted the feature/582 branch August 10, 2020 16:14
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