Closed rbecheras closed 8 years ago
Very well, I should do these tasks this week. Thanks
@deeg , sorry I missed to update the code for a while, but it's done. You can merge it now.
Please see my comment on #210 about explaining why SetContent is needed.
@rbecheras, going to close this PR. Lets finish the discussion over at #210, and we can get another PR opened pending the results.
I know the tests are a bit out of date, and I have a task to clean them up and add more, but it would be nice if you could add tests for this as part of the PR.
Once the changes are made we can get this merged in. Thanks for reporting and posting a PR.