-
Notifications
You must be signed in to change notification settings - Fork 23
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
Redesigning the workflow showing method #18
Conversation
@sagar15795 your build is failing. Please before push, run Gradle check. |
There seem to be more and more commits being pushed into this pull request. @sagar15795 you will need to tell us when to merge. It's great that you are adding all these things but it's getting too hard for me to follow all of this. Looking at the original comment in this thread there were a number of features being added/changed - are these still the things you are working on or have there been more? Anyway, at some point you need to stop adding things here, get us to check it and then (assuming it is all ok and we have merged it) you can work on more features in another pull request. |
@sagar15795 I hope your Favorite logic is finished today as we have talked today. Please Let me know for review and Stop working on this. anything has any little bug or incompleteness Please Note down and we will do that in next PR Thanks |
I have done with all checked field. @therajanmaurya @ianwdunlop I will add licence after your review. Thanks . |
@sagar15795 some functionality is missing. we will add in next PR. @ianwdunlop @stain Please Merge This PR. So @sagar15795 will finish this feature because this PR is the base PR of this feature. |
I have added ASF header in all modified or added file. @ianwdunlop @stain |
Workflow List [Done]
Detail Workflow [WIP]
Add zoom functionality to workflow image [will add it in next PR]