-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Package: vulcan-ui-material 1.15.1, example folder and readme.md may be outdated ? #2586
Comments
Hi, it could be an occurrence of the bug where Meteor exports an empty object instead of the actual export server side: #2580 |
I can have a look at this and see if the example components need to be updated for MUI 4. I am working on |
I think we could split them into a separate package maybe, but still in Vulcan core repo |
It will be great, if we can have a complete example app with material ui in the Vulcan-Starter. |
@ErikDakoda @eric-burel Im having the same problem trying to follow the material-ui example docs but...
|
Hi @webface it works in my localhost. To help you, I would need to know which version of Vulcan, Vulcan-Starter, vulcan:ui-material you are using, and if the following packages/versions are installed: meteor npm install --save @material-ui/core@4.11.0
meteor npm install --save @material-ui/icons@4.9.1
meteor npm install --save @material-ui/styles@4.10.0
meteor npm install --save react-jss@10.4.0
meteor npm install --save mdi-material-ui@6.19.0
meteor npm install --save react-autosuggest@10.0.2
meteor npm install --save autosuggest-highlight@3.1.1
meteor npm install --save react-isolated-scroll@0.1.1
meteor npm install --save react-keyboard-event-handler@1.5.4 |
@ErikDakoda Im using those packages @ latest `Package.describe({ Package.onUse(function (api) {
]); api.addFiles('lib/stylesheets/style.css'); // Here is the entry point for client & server: |
I created a home component const Home =({currentUser})=>{ addRoute({ name: 'home', path: '/', componentName: 'Home' }); still get that error. How to add components and where do I put the theme |
routes.js // Then, we add the component for what we want to add. // Next, we add the name 'movies', the path, which is the root, and the component name 'MovieList'. |
@ErikDakoda i updated the post. I am using the latest vulcan. the 2 repo install and I have now used the material ui version you listed. Still get the same result. Im trying to build a material ui project from scratch |
The sample Header component is so simple - and it only uses other components that are built into I can't tell you what's going on without seeing all of the code. Is there any way you can share a repository with the code? |
@ErikDakoda Im sure im doing something simple wrong https://github.com/webface/vulcan-logic package |
Ok I figured out to get the theme from here |
Renders with 2 warnings |
@ErikDakoda I shared the link to the repo, if you could help with this, I'd greatly appreciate it |
This is the second part of the error. It looks like something to do with SSR. req.url resolves to an object in
|
Also when I enable backoffice and view it I. get the same error but also for the SSR and req.url is an object => App running at: http://localhost:3000/ |
When I add the following to
So this is an issue with SSR. I have had so much frustration with SSR that now I have it turned off. Maybe someone who successfully uses SSR can help you. |
@webface If you are creating a new application, you might prefer using Vulcan Next instead of the "legacy" Vulcan Meteor project which is less active, and in particular often generating issue during SSR because of Meteor not being able to import some React components server-side (this very issue pushed us to accelerate the transition out of Meteor). |
@ErikDakoda @eric-burel makes sense. thanks for this. Does Vulcan Next have all the cool packages of Vulcan Meteor? I spent the whole holidays on Vulcan Meteor |
Most of them for the backend at this point, basically you can reuse Vulcan schemas to spawn a graphql API + mongo collection, and you have React hooks as well like "useMulti". |
@eric-burel I'll check it out. Question now is @ErikDakoda does Vulcan Meteor support SSR as long as youre using bootstrap? I can live with that |
I don't have any experience with bootstrap on Vulcan. |
I tried bootstrap. I had similar errors till I set ssr to true then it worked. Maybe I just needed to set ssr to true for the mat hi also. I'll try after work |
Hi,
I follow the instruction of the example folder and readme.md to use it with the package example-forum but I got this warning:
W20200614-14:27:57.302(7)? (STDERR) Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
W20200614-14:27:57.308(7)? (STDERR) in Header
I think that the warning is because the example folder and readme.md may be outdated and are not compatible with Vulcan 1.15.1.
Thank You
Adalidda
The text was updated successfully, but these errors were encountered: