Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

Free API calls and Enabling the API Explorer for calls not in Production #268

Merged
merged 13 commits into from
Dec 13, 2023

Conversation

sanjam-deriv
Copy link
Contributor

@sanjam-deriv sanjam-deriv commented Nov 29, 2023

This PR makes the following changes in the API Explore

  • User can now test API Calls without selecting them from the dropdown
  • The text box is enabled intially without any request being selected
  • Free API Calls from the backend can now be tested on the API Explorer if they are a part of the WS connection

Copy link

@review-deriv review-deriv left a comment

Choose a reason for hiding this comment

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

PR description is too short. You can use this format:

# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.

# Cards
* https://clickup.com/12345
* https://clickup.com/12346

Copy link

vercel bot commented Nov 29, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-api-docs ✅ Ready (Inspect) Visit Preview Dec 12, 2023 2:51pm

@sanjam-deriv sanjam-deriv marked this pull request as ready for review November 30, 2023 11:08
@sanjam-deriv sanjam-deriv changed the title Websocket update Free API calls and Enabling the API Explorer for calls not in Production Nov 30, 2023
@coveralls
Copy link

coveralls commented Dec 7, 2023

Pull Request Test Coverage Report for Build 7183156216

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 11 of 13 (84.62%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 97.561%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/hooks/useSubscription/index.tsx 4 5 80.0%
src/hooks/useWs/index.tsx 7 8 87.5%
Totals Coverage Status
Change from base Build 7127378057: -0.2%
Covered Lines: 1622
Relevant Lines: 1627

💛 - Coveralls

matin-deriv
matin-deriv previously approved these changes Dec 8, 2023
@amina-deriv

This comment was marked as resolved.

@sanjam-deriv sanjam-deriv merged commit 2393dd0 into binary-com:master Dec 13, 2023
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants