map-starter-kit icon indicating copy to clipboard operation
map-starter-kit copied to clipboard

Les types Typescript ne sont pas accessibles

Open E-OssO opened this issue 2 years ago • 4 comments

C'est dommage que les types de votre projets ne soient pas accessible. Alors que vous y avez apporté beaucoup d'énergie :)

let currentPopup: any = undefined;
// Serait mieux avec le bon typage :)
let currentPopup: Popup | undefined = undefined;

Important sur les objets manipulable tel que les layers, layersProperties, ...

E-OssO avatar Jun 17 '22 07:06 E-OssO

Les types de l'API sont disponibles il suffit d'ajouter ce tag en haut de vos fichiers TS :

/// <reference types="@workadventure/iframe-api-typings" />

Pour plus d'informations je vous renvois sur la documentation de l'API =) : https://workadventu.re/map-building/api-reference

Nolway avatar Jun 20 '22 17:06 Nolway

Effectivement, m'a demande n'était pas assez explicite ^^ il manque les exports de vos types pour qu'on puisse les utiliser.

Oui, ils sont bien définis et accessibles. Mais on ne peut pas les utiliser.

Comme j'en avais discuté avec David j'ai vu son issue après avoir remonté celle-ci. https://github.com/thecodingmachine/workadventure/issues/2313

Par contre, je crois que ca ne couvre pas le type ITiledMapLayer qui est dans mon cas le "principal" dans la manipulation des layers. Et aussi dans la class ITiledMapLayer le typage de l'attribut properties est surement a améliorer. Les attributs name, type, value sont tous en any. Et je pense que properties value mérite son propre type pour éviter d'avoir a énumérer sa liste de types dans chaque déclarations de fonction comme par exemple.

WA.room.setProperty avec la déclaration suivante setProperty(layerName: string, propertyName: string, propertyValue: string | number | boolean | undefined): void;

E-OssO avatar Jun 21 '22 06:06 E-OssO

Corrigé par https://github.com/thecodingmachine/workadventure/pull/2319

moufmouf avatar Jun 30 '22 07:06 moufmouf

Je réouvre, il manque la réexportation de ITiledMapLayer

moufmouf avatar Jun 30 '22 07:06 moufmouf

@moufmouf Can you tell me the status of this issue ?

Nolway avatar Nov 16 '22 15:11 Nolway

Can we close this guys?

ValdoTR avatar Feb 07 '23 14:02 ValdoTR