Skip to content

FC-0057: add params parameter to the requests#12

Merged
Faraz32123 merged 1 commit intomasterfrom
fix/add_params_to_the_request_plus_some_improvements
Aug 1, 2024
Merged

FC-0057: add params parameter to the requests#12
Faraz32123 merged 1 commit intomasterfrom
fix/add_params_to_the_request_plus_some_improvements

Conversation

@Faraz32123
Copy link
Copy Markdown
Contributor

  • add missing params parameter to the requests call
  • simplify all requests call into a single request helper

Merge checklist:
Check off if complete or not applicable:

  • Version bumped
  • Changelog record added
  • Documentation updated (not only docstrings)
  • Fixup commits are squashed away
  • Unit tests added/updated
  • Manual testing instructions provided
  • Noted any: Concerns, dependencies, migration issues, deadlines, tickets

@Faraz32123 Faraz32123 marked this pull request as ready for review August 1, 2024 13:29
@Faraz32123 Faraz32123 requested a review from Ali-Salman29 August 1, 2024 13:29
Copy link
Copy Markdown
Contributor

@taimoor-ahmed-1 taimoor-ahmed-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Faraz32123 Faraz32123 force-pushed the fix/add_params_to_the_request_plus_some_improvements branch from e0672b1 to c267878 Compare August 1, 2024 13:37
- add missing params to the requests call
- simplify all requests call into a single request helper
@Faraz32123 Faraz32123 force-pushed the fix/add_params_to_the_request_plus_some_improvements branch from c267878 to 4ab7cc8 Compare August 1, 2024 14:44
@Faraz32123 Faraz32123 merged commit 73d0467 into master Aug 1, 2024
@Faraz32123 Faraz32123 deleted the fix/add_params_to_the_request_plus_some_improvements branch August 1, 2024 14:46
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.

3 participants