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

Plugin details #98

Merged
merged 5 commits into from Jul 1, 2019
Merged

Plugin details #98

merged 5 commits into from Jul 1, 2019

Conversation

PhilippLgh
Copy link
Contributor

@PhilippLgh PhilippLgh commented Jun 27, 2019

What does it do?

  • Adds a new details tab to each config view
    • With information about the plugin such as author, signatures, location
    • A "PluginEditor" which displays the plugin source
  • Renames "nodes" to "plugins"

Any helpful background information?

From a security standpoint it is important to have information about the plugin. What does it do? Is it coming from a trusted source? Is it the correct plugin or is it pointing to a wrong local location? ...

The editor could be the foundation to write custom plugins or edit existing ones (dev only) from within Grid.

New dependencies? What are they used for?

  • codemirror (syntax highlighting and editing in browser)
  • react-codemirror2 (wrapper)
  • requires changes to Plugin in Grid

Relevant screenshots?

image

Does it close any issues?

ethereum/grid#298
ethereum/grid#287

@ryanio
Copy link
Contributor

ryanio commented Jun 28, 2019

Hm I'm getting some weird behavior. Plugin Details section is blank. Plugin Code section loads blank, but if I click on it, then it loads (but scrolled all the way to the bottom)

Screen Shot 2019-06-27 at 9 28 06 PM

Screen Shot 2019-06-27 at 9 28 16 PM

@wolovim
Copy link
Member

wolovim commented Jul 1, 2019

Fixed the empty state @ryanio. Also seeing no results in plugin details though. Investigating.

@wolovim
Copy link
Member

wolovim commented Jul 1, 2019

Ah, discovered metadata only available for remote registry plugins. I've removed the "Plugin Details" title when there is no metadata to display. If there is work to be done to display metadata of local plugins, let's open a new ticket for that.
cc/ @ryanio

@wolovim wolovim merged commit fcda3e4 into dev Jul 1, 2019
@wolovim wolovim deleted the plugin-details branch July 1, 2019 23:21
evertonfraga added a commit that referenced this pull request Jul 22, 2019
* fix: refactor VersionList (#80)

* ci: disabling nightly builds (#87)

* fix: height 100% for apps (#88)

* fix: height 100% for apps

* fix: height fix when showing url bar

* test: add timestamp of last loaded time to versionlist

* fix: do not open local server on browser (#92)

* feat: expands showOpenDialog to accept `file` or `directory` (#90)

* feat: expands showOpenDialog to accept `file` or `directory` (or `file_multiple`, `directory_multiple`)

* fix: bool console warnings

* chore: update `openDirectory` param to `pathType`

* feat: Terminal Input (#94)

* feat: terminal input

* chore: show 'No logs yet.' message instead of blank terminal

* feat: scroll through terminal input history on up arrow and down arrow

* feat: plugin list categories (#97)

* feat: custom settable flags (#95)

* feat: custom settable flags

* fix: lint

* fix: tests

* feat: persist config data (#93)

* feat: persist config data

* feat: rehydrate settings from config.json

* fix: remove redux-persist cruft

* fix: function purity

* feat: persists plugin selection

* fix: listSubheader material-ui warning (#99)

* fix: listSubheader material-ui warning

* fix: remove extraneous key

* feat: protect terminal input (#96)

* feat: protect terminal input

* chore: update label to 'Hide input'

* Plugin details (#98)

* add codemirror

* rename "nodes" to "plugins"

* feat: add plugin view

* fix: empty loads and warnings

* fix: hide title if no metadata

* chore: update theme (#101)

* chore: update theme

* fix: colors as constants

* chore: use newState event (#89)

* chore: use newState event

* chore: use newState event

* fix: run `onConnect` on `connected`

* fix: watchForPeers only on client.type === 'client'

* chore: move block subscriptions to own method to not clutter onConnect method

* chore: update pluginError event name (#102)

* chore: update pluginError event name

* fix: update removeListeners event name

* chore(deps): bump lodash.template from 4.4.0 to 4.5.0 (#103)

Bumps [lodash.template](https://github.com/lodash/lodash) from 4.4.0 to 4.5.0.
- [Release notes](https://github.com/lodash/lodash/releases)
- [Commits](lodash/lodash@4.4.0...4.5.0)

Signed-off-by: dependabot[bot] <support@github.com>

* fix: watchForPeers fix

* feat: component-scoped UI (#100)

* feat: start with component & client scope

* fix: add simplified config generator

* fix: change entry point of scoped view to config

* feat: allows nano to open grid-ui

* fix: createListeners for all plugins

* fix: disables toggle btns

* feat: start with component & client scope

* fix: add simplified config generator

* fix: change entry point of scoped view to config

* feat: allows nano to open grid-ui

* fix: createListeners for all plugins

* fix: disables toggle btns

* fix: capture flags in config.json

* feat: scoped apps rendering

* add xterm

* fix: restore toggle

* fix: remove cruft

* feat: Dark mode (#107)

* feat: first iteration of dark theme

* fix: lint rule exception

* chore: pass themeMode via props so that grid-ui can be opened with args.scope.themeMode = 'dark' or 'light'. defaults to 'dark'

* feat: open tabs dynamically (#106)

* feat: open tabs dynamically

* fix: resume state correctly

* fix: pass clientName correctly to set custom flags (#108)
evertonfraga pushed a commit that referenced this pull request Jul 26, 2019
* add codemirror

* rename "nodes" to "plugins"

* feat: add plugin view

* fix: empty loads and warnings

* fix: hide title if no metadata
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants