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

Refactored data binding to resolve [ALOY-1478], [ALOY-1480], [ALOY-1481] and [ALOY-1482] #780

Merged
merged 2 commits into from
Apr 8, 2016

Conversation

FokkeZB
Copy link
Contributor

@FokkeZB FokkeZB commented Mar 31, 2016

This is a complete refactoring of model and collection data binding.

DO NOT MERGE UNTIL WE'VE TESTED THIS INSIDE OUT... TWICE

Changes

Tests done

The following existing test apps had changes in the compiled code, which have been carefully reviewed and the app tested to still work as expected:

  • ALOY-1480 (NEW: also demonstrating ALOY-1478, ALOY-1481 and ALOY-1482)
  • ALOY-1479
  • ALOY-1477
  • ALOY-1474
  • ALOY-913
  • ALOY-895
  • ALOY-874
  • ALOY-892
  • ALOY-716
  • ALOY-713
  • ALOY-648
  • ALOY-644
  • ALOY-609
  • ALOY-594
  • ALOY-443
  • ALOY-440c (FIXED: Use ActionBar menu now Material no longer shows home icon)
  • ALOY-440 (FIXED: SQLite database no longer persist across Simulator builds)
  • ALOY-1479 (NOTE: Shows undefined for undefined attributes, just like it did for collections before 1.8.0)
  • ALOY-1477
  • ALOY-1127
  • ALOY-1033
  • ALOY-1000
  • advanced/proxy_property_requires (FIXED: Work around TIMOB-23130)

@FokkeZB
Copy link
Contributor Author

FokkeZB commented Apr 4, 2016

Restarted timed out tests

@feons
Copy link
Contributor

feons commented Apr 7, 2016

🎱

@feons
Copy link
Contributor

feons commented Apr 8, 2016

Approved.

@feons feons merged commit d62c0e6 into tidev:master Apr 8, 2016
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.

2 participants