Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed: add queryRequestTimeout to copyWith method #1468

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nestorsgarzonc
Copy link

@nestorsgarzonc nestorsgarzonc commented Nov 25, 2024

There are some simple steps to get your PR merged, that are the following:

  • Describe your PR and why a maintainer should merge it;
    The PR adds the queryRequestTimeout to the copyWith method
  • Put the same description inside the commit body otherwise if we change github hosting you application will be based on a instable source code;
  • Write the commit header in the way that it is following these simple rules [1]
  • Make sure that your PR will pass the CI
  • Wait for an review 😄 that will not happen if one of the previous step is missing.

1

@nestorsgarzonc nestorsgarzonc changed the title fix(graphql): add queryRequestTimeout parameter to copyWith method add queryRequestTimeout to copyWith method Nov 25, 2024
@nestorsgarzonc nestorsgarzonc changed the title add queryRequestTimeout to copyWith method Fix: add queryRequestTimeout to copyWith method Nov 25, 2024
@nestorsgarzonc nestorsgarzonc changed the title Fix: add queryRequestTimeout to copyWith method Fixed: add queryRequestTimeout to copyWith method Nov 25, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants