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 data sources service for improved structure and asynchronicity #337

Merge branch 'project-orion' into 301-refactor-data-sources-service-f…

1125eab
Select commit
Loading
Failed to load commit list.
Merged

Refactored data sources service for improved structure and asynchronicity #337

Merge branch 'project-orion' into 301-refactor-data-sources-service-f…
1125eab
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 12, 2024 in 1s

68.41% of diff hit (target 68.25%)

View this Pull Request on Codecov

68.41% of diff hit (target 68.25%)

Annotations

Check warning on line 22 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L22

Added line #L22 was not covered by tests

Check warning on line 31 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L29-L31

Added lines #L29 - L31 were not covered by tests

Check warning on line 50 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L50

Added line #L50 was not covered by tests

Check warning on line 54 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L54

Added line #L54 was not covered by tests

Check warning on line 78 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L78

Added line #L78 was not covered by tests

Check warning on line 126 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L126

Added line #L126 was not covered by tests

Check warning on line 136 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L134-L136

Added lines #L134 - L136 were not covered by tests

Check warning on line 145 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L145

Added line #L145 was not covered by tests

Check warning on line 153 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L151-L153

Added lines #L151 - L153 were not covered by tests

Check warning on line 171 in app/controllers/campaigns-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/campaigns-controller.js#L170-L171

Added lines #L170 - L171 were not covered by tests

Check warning on line 31 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L30-L31

Added lines #L30 - L31 were not covered by tests

Check warning on line 53 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L51-L53

Added lines #L51 - L53 were not covered by tests

Check warning on line 63 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L55-L63

Added lines #L55 - L63 were not covered by tests

Check warning on line 75 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L74-L75

Added lines #L74 - L75 were not covered by tests

Check warning on line 81 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L81

Added line #L81 was not covered by tests

Check warning on line 126 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L126

Added line #L126 was not covered by tests

Check warning on line 143 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L143

Added line #L143 was not covered by tests

Check warning on line 167 in app/controllers/data-sources-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/data-sources-controller.js#L166-L167

Added lines #L166 - L167 were not covered by tests

Check warning on line 33 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L31-L33

Added lines #L31 - L33 were not covered by tests

Check warning on line 62 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L50-L62

Added lines #L50 - L62 were not covered by tests

Check warning on line 77 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L77

Added line #L77 was not covered by tests

Check warning on line 123 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L123

Added line #L123 was not covered by tests

Check warning on line 131 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L129-L131

Added lines #L129 - L131 were not covered by tests

Check warning on line 139 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L139

Added line #L139 was not covered by tests

Check warning on line 147 in app/controllers/techniques-controller.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

app/controllers/techniques-controller.js#L145-L147

Added lines #L145 - L147 were not covered by tests