tuplejump / play-yeoman

Play + Yeoman integration sbt and play plugins
Apache License 2.0
237 stars 58 forks source link

Production mode is not working #95

Open immortalcy opened 8 years ago

immortalcy commented 8 years ago

When I run the latest release of play-yeoman in production mode I found out that it doesn't work. At first I thought that this was the same issue with #90 but now I think it is not.

After cloning into a new folder here are the commands under yo-demo folder I run: sbt npm install bower install dist

When I try opening the application [http://localhost:9000/ui/] I get the following error:

Not Found For request 'GET /ui/'

To relate this with #90 , this not an Action Not Found error but a Not Found error. If you try to open http://localhost:9000/tdemo it works normally.

If you extract and check yo-demo.yo-demo-1.0.0-assets.jar you will see that it doesn't contains any files from ui directory. Shouldn't all the files be there in their minified version, or is that a wrong .jar file I am looking at?

apatzer commented 8 years ago

+1

apatzer commented 8 years ago

@immortalcy If you copy everything from ui/dist to /public it will work in production mode. It's just a pain to need to do this each time you do a Grunt build.

immortalcy commented 8 years ago

Thanks for the tip, but yes as you said it not optimal. By the way I was trying to compare the two latest releases (0.8.1 with latest) and there doesn't seem to be any major differences. Was just wondering where the problem could be..

almothafar commented 8 years ago

+1 I have same issue here Its critical any response !? this issue been here more than 1 month.

almothafar commented 8 years ago

@immortalcy Thank you for your idea, and I made a workaround using grunt. inside copy task at the end after styles I added:

,
      public: {
        expand: true,
        cwd: 'dist',
        src: ['**'],
        dest: '../public/'
      }

And on register build task I added at the end of queue the following line: 'copy:public'

Now its automatically do the copy, this workaround must be work until this plugin got fixed.