This repository has been archived by the owner on Oct 17, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Steps
Purpose
Currently when using a
State
property with storage onApp
instead of on aView
, the app ID was accessed before it was set, causing theState
internals to look for the data in the foldertemporary/
which usually doesn't exist because at the time you will be writing into theState
the app ID will be set and it writes the value intoapp.id/
folder.This happened because when the
State
property is onApp
it is initialized beforeGTUIApp.appID
is set.Approach
I fixed this by making the
App.id
property static and forwarding it toGTUIApp.appID
first thing the program does. I have also made theGTUIApp.appID
force unwrapped optional so regression of this issue is not possible.I am open to suggestions of how to fix this differently. You can also fix this yourself in a different way without crediting, it is not that complicated.
Since it is a breaking change I still need to update the examples