Skip to content

Conversation

PaulRC-ioet
Copy link
Contributor

@PaulRC-ioet PaulRC-ioet commented Nov 5, 2020

Show read-only GUID field in customers and project sections.

Close #562

@codecov
Copy link

codecov bot commented Nov 5, 2020

Codecov Report

Merging #564 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #564   +/-   ##
=======================================
  Coverage   90.63%   90.63%           
=======================================
  Files          80       80           
  Lines        1441     1441           
  Branches      107      107           
=======================================
  Hits         1306     1306           
- Misses         94       95    +1     
+ Partials       41       40    -1     
Impacted Files Coverage Δ
.../components/technologies/technologies.component.ts 84.84% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ac7344...1cdc1ed. Read the comment docs.

@@ -1,6 +1,17 @@
<div class="container">
<form style="width: 600px;" [formGroup]="customerForm" (ngSubmit)="onSubmit(customerForm.value)">
<div class="form-group">
<div class="form-group">
Copy link
Contributor

@enriquezrene enriquezrene Nov 5, 2020

Choose a reason for hiding this comment

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

This is not going to solve the problem, what we need to do instead is to show the uuid in the table. Please add this column in the customer list, project list, project type and activities list as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Got it, Working on that!

@PaulRC-ioet PaulRC-ioet force-pushed the 562_readOnlyFieldIDCostomer branch from 588e51c to 1cdc1ed Compare November 5, 2020 19:18
@PaulRC-ioet
Copy link
Contributor Author

All the changes are done @enriquezrene , delete de input with the ids, and add those ids into the tables.

@enriquezrene enriquezrene merged commit 29cb6cc into master Nov 5, 2020
@enriquezrene enriquezrene deleted the 562_readOnlyFieldIDCostomer branch November 5, 2020 19:58
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.

Show read only GUID field in customers and project sections.
3 participants