[RFR] Use fakerest and webpack-dev-server instead of json-server and grunt-connect#625
[RFR] Use fakerest and webpack-dev-server instead of json-server and grunt-connect#625fzaninotto merged 7 commits intomasterfrom
Conversation
Makefile
Outdated
There was a problem hiding this comment.
don't forget to remove the related conf in Gruntfile
|
Awesome! |
jeromemacias
changed the title
There was a problem hiding this comment.
I don't understand what that file does?
There was a problem hiding this comment.
It's just a symlink to the fakerest node module, to be able to import it in the <script> of the demo index.
There was a problem hiding this comment.
Ah OK, that's how GitHub show symlinks, I didn't know
|
Let me add another item: -[ ] Add a task to compile and save the JS so that the entire demo can be run from a static server (think gh-pages) |
There was a problem hiding this comment.
why do you need to copy if there is a symlink?
There was a problem hiding this comment.
Because copy from example build directory to test build directory does not copy symlinks (which would be broken anyway).
jeromemacias
changed the title
|
Closes #320 |
|
Could you rebase? No need for the additional task I mentioned, we'll use something else for the demo. |
jeromemacias
changed the title
jeromemacias
force-pushed
the
fakerest
branch
from
5d36bf5 to
901546a
Compare
|
Rebased |
There was a problem hiding this comment.
Why does it change if the dataset is the same? Something's wrong.
There was a problem hiding this comment.
I don't know how it's possible to find 5 records, see http://ng-admin.marmelab.com/#/comments/list?search={%22q%22:%22be%22}&page=1
But if I search for b instead of be, 5 records are found, maybe a coincidence, but this is the only explaination I see, protractor wasn't wait for the e with the old basecode.
There was a problem hiding this comment.
OK, FakeRest doesn't handle the q search inside composite fields (the 5th comment has an author.name containing 'be'.
jeromemacias
force-pushed
the
fakerest
branch
from
901546a to
d437e9f
Compare
[RFR] Use fakerest and webpack-dev-server instead of json-server and grunt-connect
2 participants
fakerestGruntfile.js