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 project due date #3153

Merged
merged 2 commits into from
Oct 1, 2020
Merged

Add project due date #3153

merged 2 commits into from
Oct 1, 2020

Conversation

joaovitor3
Copy link
Contributor

Closes #3132

This Pull Request is part of the issue of adding and updating fields according to Organised Editing Guidelines

Add project due_date field in frontend.

@joaovitor3 joaovitor3 self-assigned this Jun 19, 2020
@joaovitor3 joaovitor3 force-pushed the feature/add-project-due-date branch from d6bf7f3 to 4e63050 Compare June 21, 2020 17:16
Copy link
Contributor

@willemarcel willemarcel left a comment

Choose a reason for hiding this comment

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

@joaovitor3 react-datepicker is a good choice. Could you try to set the locale using what is defined by the user preferences?

@willemarcel
Copy link
Contributor

@joaovitor3 try also to format the input field with the same height of the text input fields.

@joaovitor3
Copy link
Contributor Author

Thanks for the review!
I will update the PR with these changes

@joaovitor3
Copy link
Contributor Author

@willemarcel, Just updated the PR with the changes requested.

Copy link
Contributor

@pantierra pantierra left a comment

Choose a reason for hiding this comment

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

This looks good to me. But has a conflict that needs to be resolved.

@joaovitor3
Copy link
Contributor Author

Just rebased the branch, could you review it please?

@willemarcel willemarcel force-pushed the feature/add-project-due-date branch 2 times, most recently from e35d21a to 534290d Compare September 30, 2020 14:29
@willemarcel willemarcel merged commit b6643ba into develop Oct 1, 2020
@willemarcel willemarcel added this to the 4.1.10 milestone Oct 1, 2020
@willemarcel willemarcel deleted the feature/add-project-due-date branch October 22, 2020 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update fields according to Organised Editing Guidelines
3 participants