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

Remove bind references and and call _render explicitly #40

Merged
merged 3 commits into from
May 17, 2017

Conversation

agubler
Copy link
Member

@agubler agubler commented May 16, 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:

Upgrade to support changes in widget-core. 1) removal of bind property 2) change to invalidating during the properties change lifecycle.

Needs release of widget-core.

@agubler agubler requested a review from kitsonk May 16, 2017 16:48
@codecov
Copy link

codecov bot commented May 16, 2017

Codecov Report

Merging #40 into master will decrease coverage by 0.29%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #40     +/-   ##
========================================
- Coverage    98.5%   98.2%   -0.3%     
========================================
  Files          10      10             
  Lines         669     670      +1     
  Branches      158     158             
========================================
- Hits          659     658      -1     
  Misses          4       4             
- Partials        6       8      +2
Impacted Files Coverage Δ
src/harness.ts 97.14% <100%> (-1.14%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 44a1a03...0b48447. Read the comment docs.

@agubler agubler changed the title Remove bind references and explicitly emit invalidate event Remove bind references and and call _render explicitly May 17, 2017
@agubler agubler force-pushed the widget-core-upgrade branch from 7d4505c to 0b48447 Compare May 17, 2017 07:38
@agubler agubler merged commit 3b5cac5 into dojo:master May 17, 2017
@dylans dylans added this to the 2017.05 milestone May 27, 2017
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.

4 participants