thekingofcity / pravega-desktop-manager Goto Github PK

Problem description
Use electron and react to build a Pravega GUI from scratch.

Problem location
./

Suggestions for an improvement

Problem description
Users should be able to create a stream with different retention policies and scaling policies.

Problem location
src\renderer\view\connection\dialogs.tsx and relating IPC in main process.

Suggestions for an improvement

Problem description
Pravega exposes its metrics through the Micrometer Metrics. Displaying them in the dashboard will improve the user experience.

Problem location
src\renderer\view\connection\overview.tsx

Suggestions for an improvement

Problem description
Adding multiple language support will help users become familiar with Pravega quickly.

Problem location
i18n

Suggestions for an improvement
Note that Japanese and Korean are translated by machine. Any embellishment will be welcomed.

Problem description
Upgrade React and Electron to their latest.

Problem location
Mainly package.json

Suggestions for an improvement

Problem description
Users should be able to pause the read.

Problem location
src\renderer\view\connection\preview-stream.tsx

Suggestions for an improvement
Set the scopedStreams in the main process to undefined if the user clicks the pause button.

Problem description
Upgrade version to 4.

Problem location
release/app/package.json

Suggestions for an improvement

Problem description
Leverage the latest pravega nodejs feature. pravega/pravega-client-rust#382

Problem location
src\main\pravega.ts
src\renderer\views\connection\preview\preview-stream.tsx

Suggestions for an improvement

Problem description
List scopes and stream async to prevent the main process from hanging in the operation.

Problem location
src\main\pravega.ts

Suggestions for an improvement
Use the latest pravega/pravega-client-rust#382.

Problem description
Users are only allowed to read from the current tail. It will be better to enable the user to read from the head.

Problem location
src\renderer\view\connection\preview\preview-stream.tsx

Suggestions for an improvement

Problem description
Remove unnecessary dependencies to minimize the third-party usage and their licenses.

Problem location
All over the place.

Suggestions for an improvement

Problem description
electron-builder install-app-deps will set node_napi_label to electron-v16.0 because electron uses a different API than Node. The @thekingofcity/pravega package only releases pure node build.
Electron Native Node Modules

Problem location
package.json

Suggestions for an improvement
As the pure node binary works for the project (no electron-specific code is called), we may override this runtime and use the pure node binary.

Problem description
Users should be able to filter events based on regular expressions.

Problem location
src\renderer\views\connection\preview\preview-stream.tsx

Suggestions for an improvement

Problem description
Upgrade version to 3.

Problem location
release/app/package.json

Suggestions for an improvement

Problem description
Upgrade version to 2.

Problem location
release/app/package.json

Suggestions for an improvement

Problem description
Update README with new demos and screenshots.

Problem location
README.md

Suggestions for an improvement

PravegaDesktopManagerDemo.mp4

Problem description
The video is not displayed in the Readme.

Problem location
README.md

Suggestions for an improvement
<video src="https://user-images.githubusercontent.com/3353040/180732782-60c2bbf7-6c1e-413f-8af2-8894364923d7.mp4">

Problem description
Lint indent to 4. This should fix actions.

Problem location
All over the place.

Suggestions for an improvement