Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add passphrase compatibility to the hw wallet #2218

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Senyoret1
Copy link
Contributor

Changes:

  • Now it is possible to add passphrases to the hardware wallet.

This PR includes many texts, so it would be good to check them with some care, since there may be errors.

Does this change need to mentioned in CHANGELOG.md?

@gz-c gz-c requested a review from mahansky March 7, 2019 09:06
@gz-c
Copy link
Member

gz-c commented Mar 11, 2019

Will not be merged until later; this is scheduled for v2 wallet firmware

@gz-c gz-c changed the title Add passphrase compatibility Add passphrase compatibility to the hw wallet Jun 24, 2019

import { HwPassphraseDialogComponent } from './hw-passphrase-dialog.component';

describe('HwPassphraseDialogComponent', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
describe('HwPassphraseDialogComponent', () => {
describe("HwPassphraseDialogComponent", () => {

@@ -0,0 +1,14 @@
import { Component } from '@angular/core';
import { MatDialogRef } from '@angular/material/dialog';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { MatDialogRef } from '@angular/material/dialog';
import { MatDialogRef } from "@angular/material/dialog";


@Component({
selector: 'app-hw-passphrase-help-dialog',
templateUrl: './hw-passphrase-help-dialog.component.html',
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
templateUrl: './hw-passphrase-help-dialog.component.html',
templateUrl: "./hw-passphrase-help-dialog.component.html",

import { MatDialogRef } from '@angular/material/dialog';

@Component({
selector: 'app-hw-passphrase-help-dialog',
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
selector: 'app-hw-passphrase-help-dialog',
selector: "app-hw-passphrase-help-dialog",

@@ -12,6 +12,7 @@ import { HwChangePinDialogComponent } from '../hw-change-pin-dialog/hw-change-pi
import { HwRestoreSeedDialogComponent } from '../hw-restore-seed-dialog/hw-restore-seed-dialog.component';
import { Observable } from 'rxjs/Observable';
import { HwDialogBaseComponent } from '../hw-dialog-base.component';
import { HwPassphraseActivationDialogComponent } from '../hw-passphrase-activation-dialog/hw-passphrase-activation-dialog.component';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { HwPassphraseActivationDialogComponent } from '../hw-passphrase-activation-dialog/hw-passphrase-activation-dialog.component';
import { HwPassphraseActivationDialogComponent } from "../hw-passphrase-activation-dialog/hw-passphrase-activation-dialog.component";

import { HwWalletService, OperationResults } from '../../../../services/hw-wallet.service';
import { ChildHwDialogParams } from '../hw-options-dialog/hw-options-dialog.component';
import { HwDialogBaseComponent } from '../hw-dialog-base.component';
import { HwPassphraseHelpDialogComponent } from '../hw-passphrase-help-dialog/hw-passphrase-help-dialog.component';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { HwPassphraseHelpDialogComponent } from '../hw-passphrase-help-dialog/hw-passphrase-help-dialog.component';
import { HwPassphraseHelpDialogComponent } from "../hw-passphrase-help-dialog/hw-passphrase-help-dialog.component";


import { HwPassphraseActivationDialogComponent } from './hw-passphrase-activation-dialog.component';

describe('HwPassphraseActivationDialogComponent', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
describe('HwPassphraseActivationDialogComponent', () => {
describe("HwPassphraseActivationDialogComponent", () => {

import { MatDialogRef, MAT_DIALOG_DATA, MatDialog, MatDialogConfig } from '@angular/material/dialog';
import { HwWalletService, OperationResults } from '../../../../services/hw-wallet.service';
import { ChildHwDialogParams } from '../hw-options-dialog/hw-options-dialog.component';
import { HwDialogBaseComponent } from '../hw-dialog-base.component';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { HwDialogBaseComponent } from '../hw-dialog-base.component';
import { HwDialogBaseComponent } from "../hw-dialog-base.component";


console.log("Hardware wallet passphrase requested");
if (win) {
win.webContents.send('hwPassphraseRequested');
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
win.webContents.send('hwPassphraseRequested');
win.webContents.send("hwPassphraseRequested");

@@ -0,0 +1,60 @@
import { Component, Inject } from '@angular/core';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { Component, Inject } from '@angular/core';
import { Component, Inject } from "@angular/core";

fixture.detectChanges();
});

it('should be created', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
it('should be created', () => {
it("should be created", () => {

@@ -0,0 +1,25 @@
import { async, ComponentFixture, TestBed } from '@angular/core/testing';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { async, ComponentFixture, TestBed } from '@angular/core/testing';
import { async, ComponentFixture, TestBed } from "@angular/core/testing";

@@ -0,0 +1,44 @@
import { Component, OnInit, OnDestroy } from '@angular/core';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { Component, OnInit, OnDestroy } from '@angular/core';
import { Component, OnInit, OnDestroy } from "@angular/core";


@Component({
selector: 'app-hw-passphrase-dialog',
templateUrl: './hw-passphrase-dialog.component.html',
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
templateUrl: './hw-passphrase-dialog.component.html',
templateUrl: "./hw-passphrase-dialog.component.html",

@Component({
selector: 'app-hw-passphrase-dialog',
templateUrl: './hw-passphrase-dialog.component.html',
styleUrls: ['./hw-passphrase-dialog.component.scss'],
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
styleUrls: ['./hw-passphrase-dialog.component.scss'],
styleUrls: ["./hw-passphrase-dialog.component.scss"],

@@ -0,0 +1,44 @@
import { Component, OnInit, OnDestroy } from '@angular/core';
import { FormBuilder, Validators, FormGroup } from '@angular/forms';
import { MatDialogRef, MatDialog, MatDialogConfig } from '@angular/material/dialog';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { MatDialogRef, MatDialog, MatDialogConfig } from '@angular/material/dialog';
import { MatDialogRef, MatDialog, MatDialogConfig } from "@angular/material/dialog";

fixture.detectChanges();
});

it('should be created', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
it('should be created', () => {
it("should be created", () => {

}

@Component({
selector: 'app-hw-passphrase-activation-dialog',
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
selector: 'app-hw-passphrase-activation-dialog',
selector: "app-hw-passphrase-activation-dialog",


openHelp() {
this.dialog.open(HwPassphraseHelpDialogComponent, <MatDialogConfig> {
width: '450px',
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
width: '450px',
width: "450px",

@@ -0,0 +1,44 @@
import { Component, OnInit, OnDestroy } from '@angular/core';
import { FormBuilder, Validators, FormGroup } from '@angular/forms';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { FormBuilder, Validators, FormGroup } from '@angular/forms';
import { FormBuilder, Validators, FormGroup } from "@angular/forms";

import { Component, OnInit, OnDestroy } from '@angular/core';
import { FormBuilder, Validators, FormGroup } from '@angular/forms';
import { MatDialogRef, MatDialog, MatDialogConfig } from '@angular/material/dialog';
import { HwWalletService } from '../../../../services/hw-wallet.service';
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
import { HwWalletService } from '../../../../services/hw-wallet.service';
import { HwWalletService } from "../../../../services/hw-wallet.service";


ngOnInit() {
this.form = this.formBuilder.group({
passphrase: ['', Validators.required],
Copy link
Contributor

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Strings must use doublequote.

Suggested change
passphrase: ['', Validators.required],
passphrase: ["", Validators.required],

Copy link
Contributor

@mahansky mahansky left a comment

Choose a reason for hiding this comment

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

.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants