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(network): prevent sending redundant commands #229

Conversation

derevnjuk
Copy link
Member

closes #228

@derevnjuk derevnjuk added the Type: bug Something isn't working label Mar 7, 2023
@derevnjuk derevnjuk self-assigned this Mar 7, 2023
@codeclimate
Copy link

codeclimate bot commented Mar 7, 2023

Code Climate has analyzed commit 3dd16aa and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 92.3% (0.0% change).

View more on Code Climate.

src/cdp/DefaultNetwork.ts Outdated Show resolved Hide resolved
@derevnjuk derevnjuk force-pushed the fix_#228/send_only_necessary_commands_to_retrieve_the_request_post_data branch 2 times, most recently from 31b1547 to 1316158 Compare March 7, 2023 10:35
@derevnjuk derevnjuk force-pushed the fix_#228/send_only_necessary_commands_to_retrieve_the_request_post_data branch from 1316158 to f815bb8 Compare March 7, 2023 10:37
@derevnjuk derevnjuk force-pushed the fix_#228/send_only_necessary_commands_to_retrieve_the_request_post_data branch from 2c5784b to e107727 Compare March 7, 2023 10:50
@derevnjuk derevnjuk requested a review from Sija March 7, 2023 10:53
@derevnjuk derevnjuk removed the request for review from Sija March 7, 2023 11:10
@derevnjuk derevnjuk merged commit 37365fe into master Mar 7, 2023
@derevnjuk derevnjuk deleted the fix_#228/send_only_necessary_commands_to_retrieve_the_request_post_data branch March 7, 2023 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Send only necessary commands to retrieve the request post data
2 participants