Conversation
plugins/derlord/OSS.md
Outdated
{ x: 548.0447387695312, y: 2669.48876953125, z: 41.156490325927734, isBlip: true } as IShopLocation, | ||
{ x: 1960.2322998046875, y: 3742.317138671875, z: 31.343746185302734, isBlip: true } as IShopLocation, | ||
{ x: 1730.01171875, y: 6416.22021484375, z: 34.03722381591797, isBlip: true } as IShopLocation, | ||
{ x: 2555.4609375, y: 382.1643371582031, z: 107.62295532226562, isBlip: true } as IShopLocation, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a specific purpose why you used as IShopLocation
for each item? Once you declare the type of the variable in question, its children will take over the type itself.
I don't see the point in overtyping.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See in playground
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overtypes is resolved soon as well, thanks. 👍
// Used to give all the shops are different blip, just copy the line as often as you need it. | ||
export const sellersBlips = [ | ||
{ sprite: 52, color: 1, text: 'Seller Example - Index 0', scale: 1 }, // Index 0 | ||
export const coreShopLocations: IShopLocation[] = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we also define the types themselves?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unsure what this exactly means? 🤔
plugins/derlord/OSS.md
Outdated
]; | ||
const anotherExampleList = [ | ||
// { Fill me! } -> Index 1 | ||
export const ShopRegistry: Array<IShop> = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, where's the type IShop
defined?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will be resolved soon. Thanks 👍
plugins/derlord/OSS.md
Outdated
``` | ||
|
||
<img src="https://user-images.githubusercontent.com/82890183/148910952-470985fe-5fed-41ed-8b87-08c9977f71c2.png" width="500"/> | ||
<img src="https://user-images.githubusercontent.com/82890183/148910952-470985fe-5fed-41ed-8b87-08c9977f71c2.png" width="6500"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this 6500?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixd
No description provided.