Skip to content
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

Merged
merged 9 commits into from
Aug 12, 2024

Conversation

blcham
Copy link
Contributor

@blcham blcham commented Aug 8, 2024

Resolves #45

@blcham
Copy link
Contributor Author

blcham commented Aug 8, 2024

@palagdan

Note, that to overcome issues with different react versions I used:
npm install --legacy-peer-deps

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.

@palagdan
Copy link
Collaborator

palagdan commented Aug 9, 2024

@blcham

The problem was indeed in version of react. I updated it to the version 18.2.0 and everything works fine:
Screenshot from 2024-08-09 21-30-56

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.

@blcham
Copy link
Contributor Author

blcham commented Aug 10, 2024

@palagdan,

  1. is the screenshot showing how it works now? Because it looks very strange ... is there any error in console?
  2. of course I would like to remove --legacy-peer-deps as well

@blcham blcham force-pushed the 45-load-module-configuration-do-not-complete branch from 8e10804 to 7c31cd4 Compare August 10, 2024 13:10
@palagdan
Copy link
Collaborator

@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?

@blcham
Copy link
Contributor Author

blcham commented Aug 11, 2024 via email

@palagdan
Copy link
Collaborator

palagdan commented Aug 11, 2024

@blcham
I investigated this problem and here is what I found:

Before updating React to version 18:

The "Error Loading SForms..." message appears in all examples except for the Hello-World example.
When attempting to call the loading module configuration in the examples where it fails, the response from S-Forms appears valid and should work, but it does not.

Screenshot from 2024-08-11 11-59-33
In the Hello-World example, which is the only one where it works, submitting the form results in a request that is missing parameters (firstName, lastName), leading to an error.
Screenshot from 2024-08-11 11-26-02

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.

@palagdan
Copy link
Collaborator

@blcham
I also investigated the issues with the incorrect forms and found some unusual results. Initially, I tested different packages and observed varying outcomes. At first, the Hello-World example returned the correct forms, while the other examples did not. I tested with different Docker builds, including ones published a year ago. However, when I tried the latest build, all forms were correct in all examples, except for the constraint validation example, which initially returned an incorrect form but then corrected itself after a few minutes. I’m quite confused by these inconsistencies. Could we review this together?

@blcham blcham force-pushed the 45-load-module-configuration-do-not-complete branch from 3906c44 to a084570 Compare August 12, 2024 13:17
blcham added 2 commits August 12, 2024 16:20
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)
@blcham blcham merged commit e9338d5 into master Aug 12, 2024
1 check passed
@blcham blcham deleted the 45-load-module-configuration-do-not-complete branch August 12, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Loading module configuration do not complete
2 participants