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

Bugfix/typescript typings #3913

Conversation

ShikiSeiren
Copy link
Contributor

  • Updated index.d.ts typings
  • Added missing interfaces and classes for all remaining files (excluding offline)
  • Adjusted minor issues (wrong tags, inconsistent spelling...) in those files

@dsilhavy dsilhavy added this to the 4.4.1 milestone Apr 26, 2022
@dsilhavy dsilhavy modified the milestones: 4.4.1, 4.5.0 Jul 15, 2022
@dsilhavy dsilhavy modified the milestones: 4.5.0, 4.6.0 Sep 20, 2022
@dsilhavy dsilhavy modified the milestones: 4.5.1, 4.6.0 Nov 3, 2022
@dsilhavy dsilhavy modified the milestones: 4.5.2, 4.6.0 Dec 8, 2022
@dsilhavy dsilhavy modified the milestones: 4.6.0, 4.7.0 Feb 7, 2023
@michal-kobylecki-youview

Hi,
recently I've come across two methods for which TS typings do not match JS implementation:
enableText and enableForcedTextStreaming.
They both return boolean what is not included in index.d.ts:

   enableText(enable: boolean): void;
   enableForcedTextStreaming(value: boolean): void;

Can they be fixed as part of this PR or is it better to create a dedicated PR to speed it up?

…ipt-typings

# Conflicts:
#	index.d.ts
#	package-lock.json
#	package.json
#	src/dash/DashHandler.js
#	src/streaming/MediaPlayer.js
#	src/streaming/metrics/utils/DVBErrorsTranslator.js
@dsilhavy dsilhavy merged commit 0911c2d into Dash-Industry-Forum:development Mar 7, 2023

export type TrackSelectionFunction = (tracks: MediaInfo[]) => MediaInfo[];

export interface MediaPlayerClass {
Copy link

Choose a reason for hiding this comment

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

the preload-Method is Missing from this interface

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks, added here: 54612bb

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.

None yet

4 participants