Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Account for weightless tours (AIC-597) #316

Merged
merged 7 commits into from
Oct 31, 2018

Conversation

Cliabhach
Copy link
Contributor

Some ArticTour objects may not come with a valid weight property. Until we derive a more generic solution for unexpected data formation, this will prevent issues at application startup caused by not detecting this property.

sam33rdhakal
sam33rdhakal previously approved these changes Oct 30, 2018
@Cliabhach Cliabhach force-pushed the pc-account-for-weightless-tours-AIC-597 branch from 35bbef5 to e0087df Compare October 31, 2018 17:08
@Cliabhach Cliabhach force-pushed the pc-account-for-weightless-tours-AIC-597 branch from e0087df to f53dd2a Compare October 31, 2018 18:05
@Cliabhach
Copy link
Contributor Author

Oh, I see. I broke one of the tests with that update to ::enforceSanityCheck. Now that the commit has been edited to mock out that SQLiteOpenHelper, it should be able to pass on Circle CI.

@Cliabhach
Copy link
Contributor Author

And it did! @sam33rdhakal would you re-approve this?

@Cliabhach Cliabhach merged commit fcd9087 into dev Oct 31, 2018
@Cliabhach Cliabhach deleted the pc-account-for-weightless-tours-AIC-597 branch October 31, 2018 18:32
@Cliabhach
Copy link
Contributor Author

Cliabhach commented Oct 31, 2018

We'll need to make use of weight too, now that we have it coming in correctly. That'll be a separate PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants