amarbajric / EBUSA-AIM17

BPMaaS. Platform for buying, selling and reviewing processes
MIT License
2 stars 3 forks source link

Approval view #87 #90

Closed Graf-Carello closed 6 years ago

Graf-Carello commented 6 years ago

resolves #87 This is the initial implementation of the approval page. Further To-Do is to implement the actual API calls to the specific endpoints to use real data instead of test data.

amarbajric commented 6 years ago

At first glance the idea is kinda acceptable. Nevertheless, there are some aspects which need refactoring and adaptations.

1. This section should be the parent component (i.e. this is what an approver should see if he visits the tab Approval Page.) screen shot 2018-06-04 at 15 26 16

This section should be something like the approval-details page (i.e. when the approver selects one process from the list illustrated in the picture above) screen shot 2018-06-04 at 15 26 56

ToDo:

NOTE: As the process store is not yet implemented, it is only required to do the tasks mentioned above. Logic and final implementation hast to wait until the backend is ready. Best case scenario would be however, if you just take some mockup data and test it with them (just look up some tuts. on how to create mockup data in Angular)

Graf-Carello commented 6 years ago

I implemented your requested changes now and pushed them to my feature branch.

amarbajric commented 6 years ago

For me it is still incomplete. To avoid redundancy in the comments, please check my previous request (comment) on how I thought about the implementation. In short, I would suggest to have one page which lists all processes that require a review. If the approver clicks on one of the processes in the list, the application switches the page to a details page where you ONLY see the one process and can comment, approve or reject (deny) the process. Besides, relevant information about the process should also be shown there.

Nevertheless, I will approve this PR BUT it is still incomplete. Thus, as soon as the backend is providing all the necessary endpoints, this issue needs to be implemented well.