Skip to content

Conversation

@kevinjlope
Copy link
Contributor

Column of technologies add in the tablet of projects.

@kevinjlope kevinjlope force-pushed the TT-41-Save-technologies-as-part-of-project branch from e45890c to 2da046b Compare May 25, 2021 21:28
@codecov
Copy link

codecov bot commented May 26, 2021

Codecov Report

Merging #687 (19fe38b) into master (13de42e) will increase coverage by 0.73%.
The diff coverage is 93.23%.

❗ Current head 19fe38b differs from pull request most recent head 2ed3335. Consider uploading reports for the commit 2ed3335 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #687      +/-   ##
==========================================
+ Coverage   93.09%   93.82%   +0.73%     
==========================================
  Files          85      103      +18     
  Lines        1564     2042     +478     
  Branches      107      148      +41     
==========================================
+ Hits         1456     1916     +460     
- Misses         67       84      +17     
- Partials       41       42       +1     
Impacted Files Coverage Δ
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...er-management/store/customer-management.effects.ts 83.78% <14.28%> (-16.22%) ⬇️
...ared/components/input-date/input-date.component.ts 64.28% <50.00%> (-4.95%) ⬇️
.../components/project-list/project-list.component.ts 65.78% <52.17%> (-18.43%) ⬇️
...time-entries-table/time-entries-table.component.ts 61.90% <60.00%> (+6.34%) ⬆️
src/app/modules/user/store/user.reducer.ts 62.50% <62.50%> (ø)
src/app/modules/user/services/user-info.service.ts 66.66% <66.66%> (ø)
src/app/modules/user/services/user.service.ts 75.00% <75.00%> (ø)
...dules/time-entries/pages/time-entries.component.ts 85.33% <82.60%> (+2.52%) ⬆️
...es-management/store/activity-management.effects.ts 94.59% <83.33%> (-5.41%) ⬇️
... and 75 more

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 9082b90...2ed3335. Read the comment docs.

@scastillo-jp scastillo-jp changed the title TT - 41 save technologies as part of project TT-41 save technologies as part of project May 26, 2021
const addProps = btnProps.find((prop) => prop.key === this.setActive(project.status));
return { ...project, ...addProps };
});
//console.log(this.projects[0])
Copy link
Collaborator

Choose a reason for hiding this comment

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

Delete this lines


.project-list {
max-height: 300px; overflow: auto; display:inline-block; width: 100%;
height: 300%; overflow: auto; display:inline-block; width: 100%;
Copy link
Collaborator

Choose a reason for hiding this comment

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

you can use auto height: auto

export const environment = {
production: false,
timeTrackerApiUrl: 'https://timetracker-api.azurewebsites.net',
// timeTrackerApiUrl: 'https://127.0.0.1:5000',
Copy link
Collaborator

Choose a reason for hiding this comment

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

dele this comment

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

5 participants