Settings fix #387

Merged
6ea86b96 merged 2 commits from settings-fix into master 2017-07-26 13:36:07 +02:00
6ea86b96 commented 2017-07-26 10:21:03 +02:00 (Migrated from github.com)

Currently reloading the settings page blows up the app trying to access daemonSettings.max_key_fee.amount before the settings have been loaded.

Currently reloading the settings page blows up the app trying to access `daemonSettings.max_key_fee.amount` before the settings have been loaded.
kauffj commented 2017-07-26 13:36:19 +02:00 (Migrated from github.com)

Merged. Though I do support defaulting fetchable objects to undefined (or null) rather than empty objects, I think it makes the state clearer.

Merged. Though I do support defaulting fetchable objects to undefined (or null) rather than empty objects, I think it makes the state clearer.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#387
No description provided.