-
Notifications
You must be signed in to change notification settings - Fork 55
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
Import Tables From DB. #709
Comments
It's not an error, just information (you can see it's at INFO level).
Portofino looks into the data to see if maybe some string column is
actually boolean (sometimes encoded as the strings T/F or Y/N in older
database designs), however if there are too few rows to do such an analysis
it reports it.
Probably it's too detailed and should be logged at DEBUG level instead.
Il ven 17 mag 2024, 03:54 Mooaz ***@***.***> ha scritto:
… image.png (view on web)
<https://github.com/ManyDesigns/Portofino/assets/111727722/f1b263bf-df4f-4cc7-9ad7-e5093158f7b1>
from this page
—
Reply to this email directly, view it on GitHub
<#709 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHV3NSUTFUV5UINNSFFFEDZCVPMHAVCNFSM6AAAAABH3IL2P6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJWGQ4DOMZYHE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@alessiostalla |
@alessiostalla |
There's probably some other error down the line, could you send the whole
log file?
Il ven 17 mag 2024, 12:04 Mooaz ***@***.***> ha scritto:
… @alessiostalla <https://github.com/alessiostalla>
I think there are missing configuration for that.
why didn't get the tables and skip the page after click on finish button
then create pages which has cruds?!
may be it's not working or missing config if you can know what is the
missing.
Thanks.
—
Reply to this email directly, view it on GitHub
<#709 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHV3NWCVNJ4O6SINFYCBH3ZCXI4JAVCNFSM6AAAAABH3IL2P6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJXGIYDQNBYGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
And it creates some actions in action directory and didn't create any page.
|
@alessiostalla |
From the log it looks like there's a table without primary key, which is
not supported.
Portofino should detect it, issue a warning and ignore the table, but
apparently a bug crept it that some code fails because of the missing pk
before the code that would have skipped the table has a chance to run.
Il ven 17 mag 2024, 13:29 Mooaz ***@***.***> ha scritto:
… @alessiostalla <https://github.com/alessiostalla>
I think it's not missing configuration because if i downloaded a new
Portofino and tried to do the same with any existing database comes with
the same problem.
—
Reply to this email directly, view it on GitHub
<#709 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHV3NQHLVGOX4DSMJATTVDZCXS3HAVCNFSM6AAAAABH3IL2P6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJXGM4TGNBXGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@alessiostalla |
@alessiostalla |
I disappointed with this. and what is the table doesn't has PK and have composite PK??! |
why this errors appears while trying to import empty tables.....
this errors comes from this page.
@alessiostalla
The text was updated successfully, but these errors were encountered: