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

Fixed #4155 - Project Template and Task creation gets a blank screen and relationship errors in the log #4161

Closed
wants to merge 5 commits into from

Conversation

Dillon-Brown
Copy link
Contributor

@Dillon-Brown Dillon-Brown commented Aug 25, 2017

Description

When creating a project from a project template the user is able to skip entering a start date causing errors and a white page.

Motivation and Context

This change ensures that a user has entered a start date when creating a project from project templates by making it a mandatory field.

Issue reference: #4155

How To Test This

  1. Create a Project Template with some Tasks.
  2. Open action-menu and click "Create Project".
  3. In the pop-up window give the new project a name and select the check box to include the tasks.
  4. Check that without entering a date the task menu will not allow you to create a project.
  5. Check that after clicking "Create Project" a project is correctly created.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

  • My code follows the code style of this project found here.
  • My change requires a change to the documentation.
  • I have read the How to Contribute guidelines.

@Dillon-Brown Dillon-Brown added Status:Assessed PRs that have been tested and confirmed to resolve an issue by a core team member LTS labels Aug 25, 2017
Dillon-Brown and others added 4 commits August 25, 2017 06:35
@Dillon-Brown Dillon-Brown removed Status:Assessed PRs that have been tested and confirmed to resolve an issue by a core team member LTS labels Sep 27, 2017
@Dillon-Brown
Copy link
Contributor Author

Superseded by #4162

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

1 participant