dynamic-links icon indicating copy to clipboard operation
dynamic-links copied to clipboard

🐛 Bug Report:

Open theemaster opened this issue 1 year ago • 0 comments

👟 Reproduction steps

In main.js shouldn't line 60:

.join(target.fallback ?? target.default ?? '')

be targets instead of target?

I tried to create types because I am using TypeScript in my appwrite functions. My type definitions where:

interface TargetPrefs {
  appName?: string;
  appPath?: string;
  appPackage?: string;
  fallback?: string;
}

interface Targets {
  [platform: string]: string | TargetPrefs;
}

interface PathConfig {
  path: string;
  targets: Targets;
}

👍 Expected behavior

The platform definitions in the CONFIG examples do not contain an example where a target/platform contains a default property.

👎 Actual Behavior

Only targets have a property default which might be accessible.

🎲 Appwrite version

Version 1.4.x

💻 Operating system

Linux

🧱 Your Environment

appwrite 1.4.3

nodeJS using TypeScript

👀 Have you spent some time to check if this issue has been raised before?

  • [X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

theemaster avatar Oct 07 '23 22:10 theemaster