Skip to content
This repository has been archived by the owner on Feb 26, 2020. It is now read-only.

[BETA] Use persistent storage #186

Merged
merged 1 commit into from
Aug 2, 2018
Merged

[BETA] Use persistent storage #186

merged 1 commit into from
Aug 2, 2018

Conversation

amaury1093
Copy link
Contributor

Backport #177

  • Use persistent storage for webviews

  • Update .gitlab-ci.yml

enable gitsync

  • Retrieve dapp token internally

  • Remove shell_requestNewToken RPC method. The token must be retrieved internally (by importing DappsRequestsStore in Parity UI), this way we can be sure that the appId is coming from a trusted source. The token, along with the appId, are now passed as query string parameters to the dapp (and to subsequent pages during in-dapp navigation)

  • Use new version of @parity/api stripped of shell_requestNewToken methods that are now unused

  • Use synchronous blocking methods in preload.js to maximize chances of web3/parity/ethereum objects being available directly to the dapp, before its own js executes.

* Use persistent storage for webviews

* Update .gitlab-ci.yml

enable gitsync

* Retrieve dapp token internally

* Remove shell_requestNewToken RPC method. The token must be retrieved internally (by importing DappsRequestsStore in Parity UI), this way we can be sure that the appId is coming from a trusted source. The token, along with the appId, are now passed as query string parameters to the dapp (and to subsequent pages during in-dapp navigation)
* Use new version of @parity/api stripped of shell_requestNewToken methods that are now unused
* Use synchronous blocking methods in preload.js to maximize chances of web3/parity/ethereum objects being available directly to the dapp, before its own js executes.
@amaury1093 amaury1093 added the A8-backport Pull request is already reviewed well in another branch. label Aug 2, 2018
@amaury1093 amaury1093 merged commit 5e781f7 into beta Aug 2, 2018
@amaury1093 amaury1093 deleted the am-persist branch August 2, 2018 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-backport Pull request is already reviewed well in another branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants