Skip to content

Conversation

edgardavid2015
Copy link
Contributor

Problem

In the time-entries section, when there is no data, the message "The data could not be load" is displayed.
Captura de Pantalla 2021-08-06 a la(s) 12 27 58

Solution

When there is no data, the message "The data could not be load" must not be displayed, instead the message "No data available in table" should be displayed in the table section. That is the purpose of this pull request.
Captura de Pantalla 2021-08-06 a la(s) 12 19 29

@codecov
Copy link

codecov bot commented Aug 6, 2021

Codecov Report

Merging #716 (fae7093) into master (13de42e) will increase coverage by 2.00%.
The diff coverage is 96.07%.

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

@@            Coverage Diff             @@
##           master     #716      +/-   ##
==========================================
+ Coverage   93.09%   95.09%   +2.00%     
==========================================
  Files          85      106      +21     
  Lines        1564     2204     +640     
  Branches      107      168      +61     
==========================================
+ Hits         1456     2096     +640     
+ Misses         67       64       -3     
- Partials       41       44       +3     
Impacted Files Coverage Δ
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...time-entries-table/time-entries-table.component.ts 63.63% <60.00%> (+8.08%) ⬆️
...toggles/filters/feature-filter-provider.service.ts 80.00% <62.50%> (+2.22%) ⬆️
src/app/modules/user/store/user.reducer.ts 62.50% <62.50%> (ø)
...ared/components/input-date/input-date.component.ts 66.66% <66.66%> (-2.57%) ⬇️
...dules/time-entries/pages/time-entries.component.ts 85.84% <85.93%> (+3.02%) ⬆️
...project-list-hover/project-list-hover.component.ts 90.38% <92.00%> (+2.01%) ⬆️
.../components/project-list/project-list.component.ts 96.66% <93.33%> (+12.45%) ⬆️
src/app/modules/users/store/user.reducers.ts 94.73% <94.73%> (ø)
...-entries/components/calendar/calendar.component.ts 95.16% <95.16%> (ø)
... and 85 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 b135a96...afc51c1. Read the comment docs.

@edgardavid2015 edgardavid2015 force-pushed the TT-304-Handle-message-the-data-could-not-be-load branch from 8d6aac4 to ae1eaee Compare August 11, 2021 05:44
Copy link
Contributor

@jcalarcon98 jcalarcon98 left a comment

Choose a reason for hiding this comment

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

Good Job Edgar 💥

@edgardavid2015 edgardavid2015 force-pushed the TT-304-Handle-message-the-data-could-not-be-load branch from ae1eaee to afc51c1 Compare August 13, 2021 17:02
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    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

@edgardavid2015 edgardavid2015 merged commit d2fc2a0 into master Aug 13, 2021
@edgardavid2015 edgardavid2015 deleted the TT-304-Handle-message-the-data-could-not-be-load branch August 13, 2021 17:09
scastillo-jp added a commit that referenced this pull request Aug 16, 2021
scastillo-jp added a commit that referenced this pull request Aug 16, 2021
ridouku added a commit that referenced this pull request Aug 16, 2021
* fix: TT-309 Implement static packages

* fix: TT-304 Handle message: the data could not be load (#716)

* fix: TT-304 Handle message: the data could not be load

* test: TT-304 Creating an unit test to coverage the new feature

Co-authored-by: Edgar Guaman <[email protected]>

* chore(release): 1.47.1 [skip ci]nn

* Revert "fix: TT-304 Handle message: the data could not be load (#716)" (#723)

This reverts commit d2fc2a0.

* chore(release): 1.47.2 [skip ci]nn

* fix: TT-309 Implement static packages

* fix: TT-311 Can not update entries fixed (#722)

* fix: TT-311 Can not update entries fixed

* code-smell: TT-311 Code smells suggested by Sonarcloud fixed

* fix: TT-311 Fix tests that fails on different timezone

* fix: TT-311 Spelling error on method fixed

* chore(release): 1.47.3 [skip ci]nn

* fix: TT-309 Implement static packages

Co-authored-by: Edgar David Guamán <[email protected]>
Co-authored-by: Edgar Guaman <[email protected]>
Co-authored-by: semantic-release-bot <[email protected]>
Co-authored-by: Sandro Castillo <[email protected]>
Co-authored-by: Jean Carlos Alarcón <[email protected]>
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.

4 participants