Skip to content
This repository has been archived by the owner on Aug 21, 2020. It is now read-only.

Props #4

Merged
merged 3 commits into from
May 18, 2020
Merged

Props #4

merged 3 commits into from
May 18, 2020

Conversation

myobie
Copy link
Member

@myobie myobie commented May 17, 2020 •
edited
Loading

  • Defined a Props standard
    • Eventually, we will use this to provide different things like an
      application environment, context, etc
  • Provide a way to clean props (restore to minimal set)
  • Because we are using the raf now, keep track of the previous state
    at the application level (unrelated, but I'm still going to include it)

@myobie myobie self-assigned this May 17, 2020
@myobie
Keep track of previous state at the app level
9505b58 
Since we are now using the raf, the prevState should represent the
previously rendered state and not the previously mutated one
@myobie
Export Props and provide a way to produce a cleaned props
416a340
@myobie myobie changed the title Bundle up template variables into props Props May 18, 2020
@myobie myobie added merge and removed in progress labels May 18, 2020
@myobie myobie merged commit 7fad867 into master May 18, 2020
@myobie myobie deleted the props branch May 18, 2020 11:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@myobie myobie

Labels
merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@myobie