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

Add optional deadline feature #45

Closed
wants to merge 2 commits into from

Conversation

hlavu
Copy link

@hlavu hlavu commented Oct 9, 2021

Hi @vaishnav-sh, I have added the deadline feature for the app. Please take a look and let me know if you want any change.
Here are some screen shots of the app. #29
image
image

@hlavu hlavu changed the title Add deadline feature Fixes #29: Add optional deadline feature Oct 9, 2021
@hlavu
Copy link
Author

hlavu commented Oct 12, 2021

Hi @vaishnav-sh, can you please review my code and give some feedbacks if there are some needs to change?

@hlavu hlavu changed the title Fixes #29: Add optional deadline feature Add optional deadline feature Oct 12, 2021
@vaishnav-sh
Copy link
Owner

hey! I have seen your email and I'm sorry it took some time for me to reply as I was ill. This feature can be really handy but UI wise could you make some improvements?

@hlavu
Copy link
Author

hlavu commented Oct 15, 2021

Hi @vaishnav-sh, I am so sorry to hear that. Hope you feel better soon.
Here is my new design, please take a look and tell me if you have any suggestions.

image

After the Add due date btn is clicked.

image

@hlavu
Copy link
Author

hlavu commented Oct 27, 2021

Hi @vaishnav-sh, please give feed back for this PR.

@vaishnav-sh
Copy link
Owner

Sorry! This PR cannot be merged as the UI doesn't match the look of the project. Feel free to raise any other issue and I'll be happy to assign you. Thank you for contributing! but for now I will close this issue.

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.

None yet

2 participants