Wasp•2y ago

When i try to install opensaas and when i try to wasp db migrate-dev i get this

[ Wasp ] ext-src/admin/elements/charts/BarChart.tsx(95,9): error TS2353: Object literal may only specify known properties, and 'radius' does not exist in type '{ strokeColor?: string; strokeWidth?: number; fillColors?: string[]; shape?: ApexMarkerShape; offsetX?: number; offsetY?: number; customHTML?(): any; onClick?(): void; }'.

--- [Error] Your wasp project failed to compile: -------------------------------- SDK build failed with exit code: 2

--- [Error] Compilation of wasp project failed: --------------------------------1 errors found

kapa.ai•7/21/24, 9:13 PM

Hi @Sava. I'm kapa.ai, your friendly support bot. I will try to answer your question by searching through the knowledge base. :wave:

kapa.ai•7/21/24, 9:14 PM

----------------------
The error message you're seeing is related to TypeScript and it's indicating that the 'radius' property does not exist in the type you're trying to use it with. This error is not directly related to Wasp or the wasp db migrate-devwasp db migrate-dev command, but rather to the TypeScript code in your project.

The error is coming from the file ext-src/admin/elements/charts/BarChart.tsxext-src/admin/elements/charts/BarChart.tsx at line 95, column 9. You're trying to use a 'radius' property that the type does not recognize.

To fix this issue, you should check the type definition and make sure 'radius' is a valid property for that type. If 'radius' is not a valid property, you should remove it or replace it with a valid property. If 'radius' should be a valid property, then you might need to update the type definition to include 'radius'.

Unfortunately, without the specific TypeScript code and type definitions, I can't provide a more specific solution. I recommend referring to the TypeScript documentation or the documentation of the library you're using for creating charts for more information.

Please note that this error is preventing your Wasp project from compiling, so you'll need to fix it before you can successfully run wasp db migrate-devwasp db migrate-dev or any other Wasp commands.
----------------------
Note: you can ask me a follow up question by @ mentioning me again :speech_balloon:
----------------------

kapa.ai•7/21/24, 9:14 PM

Sorry @Sava, I seem to be having a bit of trouble. Don't worry though, I've pinged my support and they will try to fix me up! :technologist:

Perhaps try again in 15mins?

evilresident•7/21/24, 9:16 PM

message.txt2.53KB

SavaOP•7/21/24, 9:19 PM

Thank you, this works, but why is it happening?

evilresident•7/21/24, 9:20 PM

Honestly i only just setup 5 mins ago and used chatgpt to fix the code

SavaOP•7/21/24, 9:21 PM

chatgpt comes to rescue! But now i am getting just white screen like the app is broken, did you encounter this error?

evilresident•7/21/24, 9:22 PM

post pics?

SavaOP•7/21/24, 9:23 PM

Here, and i there is noting in console that would suggest where the problem is

FireShot_Capture_025_-_My_Open_SaaS_App_-_localhost.png

evilresident•7/21/24, 9:26 PM

try restart, honestly i started this software tonight

MEE6•7/21/24, 9:26 PM

Wohooo @evilresident, you just became a Waspeteer level 1!

SavaOP•7/21/24, 9:28 PM

Unfortunately this did not fix the problem, but tnx for the help. Hope you enjoy wasp

SSava Here, and i there is noting in console that would suggest where the problem is

IamIconLiving•7/22/24, 7:16 AM

If you have white page and absolutely no errors in console, might be something wrong with routing

SavaOP•7/22/24, 7:51 AM

Yea but why would that happen on fresh install...

MEE6•7/22/24, 7:51 AM

Wohooo @Sava, you just became a Waspeteer level 2!

miho•7/22/24, 8:15 AM

Ok, so you setup a new project with Open SaaS

wasp new <app-name> -t saaswasp new <app-name> -t saas and this just happened without any changes from your side? @Sava

miho•7/22/24, 8:19 AM

I'm getting the same error when I try the open-saas template @Vinny (@Wasp)

SavaOP•7/22/24, 8:25 AM

Hi Miho, I just used "wasp new" and then i choose saas as number 3. I did no change in template

miho•7/22/24, 8:34 AM

Yep, I've tracked the problem down to apexchartsapexcharts the library we are using for charts. They released a new version where they removed the radiusradius property: https://github.com/apexcharts/apexcharts.js/releases/tag/v3.51.0 and this is the line https://github.com/apexcharts/apexcharts.js/commit/b31f7e13b13bf3f35e2b77df12a7445aa7071c71#diff-1156359dc5238d9290bf8627dd0d6607f612e09ef7bdc89eb1598205eab1825bL834

So, in a nutshell - just remove the radiusradius property and it'll work.

We need to update the template not to include the radiusradius property

GitHub

Release v3.51.0 · apexcharts/apexcharts.js

What's Changed

More marker shapes

replace margin with offsetX/Y in radialbar datalabels - https://apexcharts.com/docs/options/plotoptions/radialbar#barLabels
fix: line/realtime test by @...

GitHub

fixes #4569; more marker shapes · apexcharts/apexcharts.js@b31f7e1

Mmiho Yep, I've tracked the problem down to `apexcharts` the library we are using for ...

IamIconLiving•7/22/24, 8:39 AM

And this didn’t give any errors in the browser console?

miho•7/22/24, 8:39 AM

It couldn't becuase the project wouldn't even pass the Typescript compile step so the app couldn't run

IamIconLiving•7/22/24, 8:42 AM

Ah ok but you got an error somewhere? Wasp logs?

miho•7/22/24, 8:42 AM

In the terminal we get this (exactly what @Sava posted)

IamIconLiving•7/22/24, 8:43 AM

Ah yes lol I missed it entirely…

miho•7/22/24, 8:43 AM

It's confusing for new users to get errors if they just started - I guess it causes the feeling of "I did something wrong" since you just started. But it was a sneaky dependency update i.e. Apexcharts not honoring semantic versioning.

IamIconLiving•7/22/24, 8:44 AM

Should probably lock deps versions to exact ones in templates to avoid these surprises

miho•7/22/24, 8:53 AM

Yep, that's one of the suggestions I put down in this issue: https://github.com/wasp-lang/open-saas/issues/241

It's annoying to get these little surprises every now and then

but it will happen since we depend on different libraries

GitHub

Update `BarChart` to work with latest `apexcharts` update · Issue #...

Users reported in Discord Typescript errors when using the latest Open SaaS: It's due to ApexCharts library we use for our charts made a breaking change in their types (removed the radius prope...

Mmiho Yep, that's one of the suggestions I put down in this issue: https://github.com/...

IamIconLiving•7/22/24, 9:12 AM

it shouldn't happen if you lock by installing the exact version

npm install apexcharts -E

npm install apexcharts -E

miho•7/22/24, 9:16 AM

Nice command, I didn't know about the -E-E option

The issue with exact versions is that you miss out on new non-breaking changes versions which could contain nice improvements. If libs follow semantic versioning (as they should be) then you get new stuff quicker instead of waiting for us to update the exact version, so that's the upside of the current approach that I personally like

ApexCharts obviously did a boo boo here with their version bump being minor IMHO

IamIconLiving•7/22/24, 9:20 AM

for sure, I get that
but I think you are in the business of building templates - something that should be built once and last forever, until you say "yeah this lib can do this now, let's update the template" or break it intentionally

otherwise this results in your templates being exposed to things like this and therefore not reliable, you never know if your template will work or not

it won't be the last time somebody doesn't respect the semver, me and you we know this

miho•7/22/24, 9:21 AM

Fair point

this is something we'll have to decide on in the team for sure

JudeJeremy•7/22/24, 9:32 AM

wait so where do i find the radius property??

JudeJeremy•7/22/24, 9:36 AM

nvm found it

miho•7/22/24, 9:39 AM

Did it work as expected after you removed the radiusradius property?

JudeJeremy•7/22/24, 9:39 AM

im running wasp start right now

JudeJeremy•7/22/24, 9:41 AM

yep it works as expected

Mmiho Did it work as expected after you removed the `radius` property?

Vinny•7/22/24, 9:53 AM

Ok @JudeJeremy @Sava @evilresident I've updated the template with radiusradius removed from the BarChart as it wasn't really being used anyway

Vinny•7/22/24, 9:53 AM

Sorry for the issue and thanks for reporting it!

IIamIconLiving for sure, I get that but I think you are in the business of building templates -...

Vinny•7/22/24, 9:58 AM

The template ships with a package.json and the versions. When I pull a fresh, new template with wasp new -t saaswasp new -t saas I get apex charts v 3.41.0

Vinny•7/22/24, 9:59 AM

so I'm assuming people who have this problem ran

npm install

npm install

instead of just

wasp start

wasp start

-- wdyt @miho ?

evilresident•7/22/24, 10:02 AM

Forgive me for the silly question, is this a single tenent app or multi tenent, i want a multi tenant with a single db for large companies (say 50 users) and to use and a shared db for 10 users per company?

Eevilresident Forgive me for the silly question, is this a single tenent app or multi tenent, ...

Vinny•7/22/24, 10:06 AM

You can definitely easily make your app multi tenancy based on the template

VVinny You can definitely easily make your app multi tenancy based on the template

evilresident•7/22/24, 10:07 AM

Could you possible help setup this or havesome sort of demo/tutorial?

Eevilresident Could you possible help setup this or havesome sort of demo/tutorial?

Vinny•7/22/24, 10:08 AM

an easy start would be to add a Company or Organization database entity, like this for example:

model Company {
  id          String         @id @default(uuid())
  name        String
  createdAt   DateTime       @default(now())
  users       User[]
}

model Company {
  id          String         @id @default(uuid())
  name        String
  createdAt   DateTime       @default(now())
  users       User[]
}

Vinny•7/22/24, 10:08 AM

and then you allow certain actions for users associated with different companies

Vinny•7/22/24, 10:09 AM

at the moment though, you'll have to use the same DB. see here: https://github.com/wasp-lang/wasp/issues/2129

evilresident•7/22/24, 10:09 AM

Thank you Vinnie

Eevilresident Thank you Vinnie

Vinny•7/22/24, 10:09 AM

you're welcome

VVinny you're welcome

evilresident•7/22/24, 10:12 AM

One more question, if i have these users for differernt companies, is there a way to limit what services they have for example com1 uses app1 app2 app3, and company2 only has app1. also how do we create these pages after user logs in? im new so sorry for silly questions

Eevilresident One more question, if i have these users for differernt companies, is there a wa...

Vinny•7/22/24, 10:13 AM

Read this and let me know if this helps: https://docs.opensaas.sh/guides/authorization/

OpenSaaS.sh

Authorization

Open SaaS is a free, open-source, full-stack SaaS starter kit for React + NodeJS.

When i try to install opensaas and when i try to wasp db migrate-dev i get this

Similar Threads

Similar Threads

Similar Threads