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

fix: handling partial error in a batch for reddit destination #3935

Merged
merged 4 commits into from
Dec 13, 2024

fix: minor change

568f72d
Select commit
Loading
Failed to load commit list.
Merged

fix: handling partial error in a batch for reddit destination #3935

fix: minor change
568f72d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 12, 2024 in 0s

26.66% of diff hit (target 90.48%)

View this Pull Request on Codecov

26.66% of diff hit (target 90.48%)

Annotations

Check warning on line 16 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L15-L16

Added lines #L15 - L16 were not covered by tests

Check warning on line 19 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L18-L19

Added lines #L18 - L19 were not covered by tests

Check warning on line 25 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L25

Added line #L25 was not covered by tests

Check warning on line 30 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L29-L30

Added lines #L29 - L30 were not covered by tests

Check warning on line 54 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L54

Added line #L54 was not covered by tests

Check warning on line 58 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L58

Added line #L58 was not covered by tests

Check warning on line 85 in src/v0/destinations/reddit/networkHandler.js

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/v0/destinations/reddit/networkHandler.js#L85

Added line #L85 was not covered by tests