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

Remove UI related settings from CLI #8783

Merged
merged 17 commits into from Jun 6, 2018
Merged

Remove UI related settings from CLI #8783

merged 17 commits into from Jun 6, 2018

Conversation

sorpaas
Copy link
Collaborator

@sorpaas sorpaas commented Jun 4, 2018

rel #8649. This removes several stuff:

  • Remove embeddable settings in dapps that allow it to be embedded in an iframe. This is not used by UI any more, and IMO it's unsafe.
  • Remove all CLI UI settings (move then to legacy), except ui-path -- this is actually not really related to UI. It is used for signer authcode storing.
  • Deprecation warnings are removed.
  • UI endpoints are removed. Please use Add dapp-wallet to built-in dapps; load it from fs parity-js/shell#126 branch to test integration.

@sorpaas sorpaas added A0-pleasereview 🤓 Pull request needs code review. M7-ui labels Jun 4, 2018
@sorpaas sorpaas added this to the 1.12 milestone Jun 4, 2018
@5chdn 5chdn mentioned this pull request Jun 4, 2018
5 tasks
fetch: F,
pool: CpuPool,
) -> Self {
let fetch_control = FetchControl::default();
let errors = Errors { embeddable_on };
let errors = Errors { };
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could be static methods instead now.

Generated token:
{}
"#,
colored(code)
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we still need it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I think we still need it. This token is needed to connect to Parity UI and for times users still need to run parity signer new-token,

@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 6, 2018
@debris debris merged commit a519044 into master Jun 6, 2018
@debris debris deleted the sorpaas/ui-removal branch June 6, 2018 08:05
@5chdn 5chdn added the B7-releasenotes 📜 Changes should be mentioned in the release notes of the next minor version release. label Jun 6, 2018
ordian added a commit to ordian/parity that referenced this pull request Jun 6, 2018
…rp_sync_on_light_client

* 'master' of https://github.com/paritytech/parity:
  Remove UI related settings from CLI (openethereum#8783)
  Remove windows tray and installer (openethereum#8778)
  docs: add changelogs for 1.10.6 and 1.11.3 (openethereum#8810)
  Fix ancient blocks queue deadlock (openethereum#8751)
  Disallow unsigned transactions in case EIP-86 is disabled (openethereum#8802)
  Fix evmbin compilation (openethereum#8795)
  Have space between feature cfg flag (openethereum#8791)
  rpc: fix address formatting in TransactionRequest Display (openethereum#8786)
dvdplm added a commit that referenced this pull request Jun 7, 2018
…eric

* origin/master:
  ethcore: fix ancient block error msg handling (#8832)
  CI: Fix docker tags (#8822)
  parity: fix indentation in sync logging (#8794)
  Removed obsolete IpcMode enum (#8819)
  Remove UI related settings from CLI (#8783)
  Remove windows tray and installer (#8778)
  docs: add changelogs for 1.10.6 and 1.11.3 (#8810)
  Fix ancient blocks queue deadlock (#8751)
  Disallow unsigned transactions in case EIP-86 is disabled (#8802)
  Fix evmbin compilation (#8795)
  Have space between feature cfg flag (#8791)
  rpc: fix address formatting in TransactionRequest Display (#8786)
  Conditionally compile ethcore public test helpers (#8743)
  Remove Result wrapper from AccountProvider in RPC impls (#8763)
  Update `license header` and `scripts` (#8666)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. B7-releasenotes 📜 Changes should be mentioned in the release notes of the next minor version release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants