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

[API] adding stream_types object and tag to forgetAll api #269

Conversation

utkarsha-deriv
Copy link
Contributor

This PR adds stream_types to api.deriv.com. Stream_types was initially available on developers.binary.com for forgetAll api call. On clicking the stream_types tag we get a box describing stream_types as well as a JSON object upon clicking the source button inside stream_types object. The PR covers implementation of these features as well as test cases.

Api.deriv.com
Screenshot 2023-12-05 at 11 52 34 AM

Developers.binary.com
image

Copy link

vercel bot commented Dec 5, 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 13, 2023 8:12am

@coveralls
Copy link

coveralls commented Dec 7, 2023

Pull Request Test Coverage Report for Build 7192396494

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.

  • 31 of 33 (93.94%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 97.596%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/features/Apiexplorer/Schema/RecursiveContent/StreamTypesObject/index.tsx 13 15 86.67%
Totals Coverage Status
Change from base Build 7127378057: -0.1%
Covered Lines: 1649
Relevant Lines: 1654

💛 - Coveralls

matin-deriv
matin-deriv previously approved these changes Dec 8, 2023
sanjam-deriv
sanjam-deriv previously approved these changes Dec 8, 2023
@sanjam-deriv sanjam-deriv merged commit 6e2523e 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