-
Notifications
You must be signed in to change notification settings - Fork 0
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
[#45] Initial try to fix module configuration not loading #46
Conversation
Note, that to overcome issues with different react versions I used: I did not investigate much, but I guess it won't be possible to fix build without upgrading react version of s-pipes-editor-ui. Thus I would do small investigation how to upgrade to react 18 and at least list the issues here. |
The problem was indeed in version of react. I updated it to the version 18.2.0 and everything works fine: However, I don't think --legacy-peer-deps flag is a good solution. While investigating the dependency issues, I discovered that many dependencies haven't been updated in years. Updating to version 18.2.0 with this flag feels like a temporary fix rather than a long-term solution. I've created another ticket to address the old dependencies and look for alternative libraries. |
|
8e10804
to
7c31cd4
Compare
@blcham |
test it
…On Sat, 10 Aug 2024 at 17:17, Daniil Palagin ***@***.***> wrote:
@blcham <https://github.com/blcham>
No, there are no errors in the console. I noticed that you changed a
dependency for s-forms. Could the problem be related to that?
—
Reply to this email directly, view it on GitHub
<#46 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA6HK7IRPTB4YE4UENJWAB3ZQYVH3AVCNFSM6AAAAABMGXOMB6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSGE4DGNZVGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@blcham Before updating React to version 18:The "Error Loading SForms..." message appears in all examples except for the Hello-World example.
After updating react to version 18:The infinite "Loading SForms" error no longer occurs, but the form is still not working correctly as it fails to send the parameters in the request. |
@blcham |
Add s-forms css as `import "@kbss-cvut/s-forms/css";` does not work properly in the project
3906c44
to
a084570
Compare
CSS files were not loaded in docker image. It is not well solved but could be solved when switching to different build tools (e.g. webpack --> vite)
Resolves #45