Skip to content
This repository has been archived by the owner on Jul 30, 2018. It is now read-only.

Initial Application State Stores Implementation #137

Merged
merged 32 commits into from
Nov 7, 2017

Conversation

agubler
Copy link
Member

@agubler agubler commented Oct 20, 2017

Type: feature

The following has been addressed in the PR:

  • There is a related issue
  • All code matches the style guide
  • Unit or Functional tests are included in the PR

Description:

Adds an initial state store to be build reactive dojo 2 applications. See README for more details.

Replaces the existing stores functionality

Resolves #136

@agubler agubler self-assigned this Oct 20, 2017
@dylans dylans added this to the 2017.10 milestone Oct 20, 2017
@kitsonk kitsonk modified the milestones: 2017.10, beta.4 Oct 30, 2017
Copy link
Member

@kitsonk kitsonk left a comment

Choose a reason for hiding this comment

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

Feels like a good enough of a base to iterate on... I really do think minimal JSDoc would be really helpful in consuming the API though.

@agubler
Copy link
Member Author

agubler commented Nov 6, 2017

@kitsonk I'll follow up with some JSDoc for the public API

@agubler agubler merged commit 63a6961 into dojo:master Nov 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants