Skip to content
This repository has been archived by the owner on Dec 7, 2021. It is now read-only.

feat: Save partial project progress during project creation #769

Merged
merged 3 commits into from
Apr 17, 2019

Conversation

wbreza
Copy link
Contributor

@wbreza wbreza commented Apr 16, 2019

This adds functionality to persist partial project information when creating a new project. Right now when creating a new connection inline within the create project flow and returning to the create project screen your partial project information is lost. Partial form progress is now saved into local storage and bound when returning to the form.

Resolves #758

@wbreza wbreza changed the base branch from master to develop April 16, 2019 20:12
Copy link
Contributor

@PIC123 PIC123 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

// If we are creating a new project check to see if there is a partial
// project already created in local storage
if (this.props.match.url === "/projects/create") {
const projectJson = localStorage.getItem(projectFormTempKey);
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we ever need to clear this? Perhaps after it gets created?

Copy link
Collaborator

Choose a reason for hiding this comment

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

nvm, just saw that down below 🙂

@pjlittle
Copy link
Collaborator

LGTM - this was annoying behavior. nice fix.

@wbreza wbreza merged commit 5574cd9 into develop Apr 17, 2019
wbreza added a commit that referenced this pull request Apr 29, 2019
This adds functionality to persist partial project information when creating a new project. Right now when creating a new connection inline within the create project flow and returning to the create project screen your partial project information is lost. Partial form progress is now saved into local storage and bound when returning to the form.

Resolves #758
@PIC123 PIC123 deleted the wabrez/save-project-progress branch August 20, 2019 18:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants