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

Refactored recent activity service for improved structure and asynchronicity #320

Open
wants to merge 12 commits into
base: project-orion
Choose a base branch
from

Conversation

vsun757
Copy link

@vsun757 vsun757 commented Oct 26, 2023

No description provided.

@vsun757 vsun757 requested a review from ElJocko October 26, 2023 14:11
@sonarcloud
Copy link

sonarcloud bot commented Oct 26, 2023

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 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Attention: 97 lines in your changes are missing coverage. Please review.

Comparison is base (505133c) 67.68% compared to head (5fbadc9) 73.05%.
Report is 297 commits behind head on project-orion.

Files Patch % Lines
app/services/recent-activity-service.js 9.34% 97 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##           project-orion     #320      +/-   ##
=================================================
+ Coverage          67.68%   73.05%   +5.36%     
=================================================
  Files                138      154      +16     
  Lines              17692    18446     +754     
  Branches            2233     3043     +810     
=================================================
+ Hits               11975    13475    +1500     
+ Misses              5710     4964     -746     
  Partials               7        7              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vsun757 vsun757 closed this Dec 13, 2023
@vsun757 vsun757 reopened this Dec 13, 2023
@vsun757 vsun757 closed this Dec 13, 2023
Copy link

sonarcloud bot commented Dec 13, 2023

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

1 New issue
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@vsun757 vsun757 reopened this Feb 15, 2024
Copy link

sonarcloud bot commented Feb 15, 2024

Quality Gate Passed Quality Gate passed

Issues
1 New issue

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

* Add new relationships-repository module
* Refactor functional to OOP
* Refactor callbacks to promises
Resolved the following linting errors:
* Error:    6:9   error  'BadlyFormattedParameterError' is assigned a value but never used  no-unused-vars
* Error:    6:54  error  'DuplicateIdError' is assigned a value but never used              no-unused-vars
* Error:   26:5   error  Expected 'this' to be used by class method '_buildQuery'           class-methods-use-this
* Error:   55:5   error  Expected 'this' to be used by class method '_buildAggregation'     class-methods-use-this
* Error:   71:5   error  Expected 'this' to be used by class method '_addLookupStages'      class-methods-use-this
* Error:   92:5   error  Expected 'this' to be used by class method '_applyFilters'         class-methods-use-this
Resolved the following linting errors in pagination-async.js:
* Error:    27:5   error  Expected blank line between class members               lines-between-class-members
* Error:    45:5   error  Expected blank line between class members               lines-between-class-members
* Error:   144:35  error  Unexpected parentheses around single function argument  arrow-parens
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