Skip to content

Conversation

gcutrini
Copy link
Member

@gcutrini gcutrini commented Aug 24, 2017

VenueRoom JSON data update decoding failed when floor key was missing.
This PR makes floor optional while processing VenueRoom data update.

@gcutrini gcutrini requested a review from smarcet August 24, 2017 22:15
Copy link
Member

@smarcet smarcet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fine with me

@gcutrini gcutrini merged commit c5baeda into develop Aug 29, 2017
@gcutrini gcutrini deleted the feature/13204 branch August 29, 2017 19:45
@gcutrini gcutrini mentioned this pull request Sep 15, 2017
gcutrini added a commit that referenced this pull request Sep 19, 2017
* [13091] [13092] Move schedule from attendee to member (#111)

* [CoreSummit] [13091] Moves schedule from attendee to member model.

* [CoreSummit] [13091] Fixes core data relationships types.

* [CoreSummit] [13091] Fixes member json parsing.

* [CoreSummit] [13091] Updates unit tests.

* [CoreSummit] [13091] Changes member schedule endpoints.

* [iOS] [13091] Moves schedule from attendee to schedule.

* [iOS] [13092] Fixes member schedule UI logics.

* [iOS] [13092] Fixes attendee order confirmation message.

* [CoreSummit] [13204] Fixes VenueRoom data update. (#112)

* [CoreSummit] [13204] Adds failing VenueRoom dataupdate unit test.

* [CoreSummit] [13204] Makes VenueRoom floor optional.

* [CoreSummit] [13224] Fixes empty schedule on Member managed object save.

* [iOS] [13232] Always checks for latest summit on load.

* [13258] Fixes schedule dates calculations using summit time zone.

* [13269] Updates aerogear-ios-http dependency.

OpenStack-mobile/aerogear-ios-http@60ed79d

* Version bump.

* [13258] Fixes tokens expiration date truncation on keychain save.

OpenStack-mobile/aerogear-ios-oauth2@4bea335
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