Melvin-Abraham/Google-Assistant-Unofficial-Desktop-Client

Switch from grpc to grpc-js

Open

#396 geöffnet am 25. Mai 2021

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (3.511 Stars) (407 Forks)batch import
google-assistant-sdkhelp wanted✨ enhancement

Beschreibung

The gRPC library is deprecated and the app should be switched to gRPC-js. This will help to keep Electron up-to-date making performance and stability better. Since grpc-js is a drop-in replacement, I can create a PR for that if we are interested in this.

Contributor Guide

Tech Stack
javascriptelectronnodejsgrpc
Domain
desktopperformance
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Node.jsElectrongRPC concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Search for usage of the deprecated `grpc` package in package.json and source files. Identify all imports and client/server code that rely on it. The new `@grpc/grpc js` package is a drop in replacement, so expect minimal API changes. Check for any compatibility notes in the repository's gRPC setup files.