Skip to content

Video #7

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

Merged
merged 4 commits into from
Jul 24, 2019
Merged

Video #7

merged 4 commits into from
Jul 24, 2019

Conversation

sanjeevghimire
Copy link
Contributor

  • Added option to choose between WML API versus DB2 API

@@ -70,7 +71,7 @@ export class LoadDataComponent implements OnInit {
this.model = new HouseInfo('', 'Select BldgType', 'Select HouseStyle', '', '', '', '', '', '', 'Select KitchenQual', '', 'Select Heating', 'Select HeatingQC', 'Select CentralAir', 'Select Electrical', 'Select RoofStyle', 'Select ExterCond', 'Select Foundation', 'Select BsmtCond', '', 'Select PoolQC', '', 'Select FireplaceQu', 'Select GarageType', 'Select GarageFinish', '', 'Select GarageCond', 'Select Fence', '', '', '');

this.addressModel = new AddressInfo('', ' ', '', '', '', '');

Copy link
Member

Choose a reason for hiding this comment

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

Did you want this space?

@@ -157,7 +158,7 @@ export class LoadDataComponent implements OnInit {
validateInputs(){
this.validate_inputs = true;
if(this.model.bldgType.search('Select') != -1){ this.errBldgType= true; this.validate_inputs=false; }
else{ this.errBldgType= false; }
else{ this.errBldgType= false; }
Copy link
Member

Choose a reason for hiding this comment

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

trailing white space

export class HouseInfo {
constructor(
public lotArea:string,
public lotArea: string,
Copy link
Member

Choose a reason for hiding this comment

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

extra space here

@scottdangelo scottdangelo merged commit cf28118 into master Jul 24, 2019
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.

2 participants