-
Notifications
You must be signed in to change notification settings - Fork 9
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
Courses created twice #496
Comments
This needs an update to the behavior of the 'save' action when first saving a new course. |
Please run |
Thanks Cobey,
The first time I ran this, I got the following message/s:
(env)
alex@alex-ThinkPad-X260:~/data/Digital-Campus/development/mpowering/django-mpowering$
make vue-deps
Makefile:77: warning: overriding recipe for target 'clean'
Makefile:61: warning: ignoring old recipe for target 'clean'
cd ./vue && yarn install
yarn install v0.27.5
[1/4] Resolving packages...
[2/4] Fetching packages...
error webpack@2.6.1: The engine "node" is incompatible with this module.
Expected version ">=4.3.0 <5.0.0 || >=5.10".
error Found incompatible module
info Visit https://yarnpkg.com/en/docs/cli/install for documentation
about this command.
Makefile:13: recipe for target 'vue-deps' failed
make: *** [vue-deps] Error 1
So then I upgraded node (the default node version in the Ubuntu repos is
clearly a very old version now) and ran the command again. This time I
got the message:
(env)
alex@alex-ThinkPad-X260:~/data/Digital-Campus/development/mpowering/django-mpowering$
make vue-dep
Makefile:77: warning: overriding recipe for target 'clean'
Makefile:61: warning: ignoring old recipe for target 'clean'
make: *** No rule to make target 'vue-dep'. Stop.
So seems like it's still not run correctly, but I'm not sure what to try
next - perhaps I'm not running this from the right ORB repo directory?
Any help much appreciated,
Cheers,
Alex
…On 27.07.2017 21:28, Cobey Potter wrote:
Please run |make vue-deps| to ensure you have correct dependencies
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#496 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAVDVb1qgJDnd_JJs_MgW8Lw1QJ4jsisks5sSNbfgaJpZM4OX-QH>.
--
Alex Little
Director, Digital Campus
alex@digital-campus.org
http://digital-campus.org
mobile (UK): +44 7548 236 650
mobile (Finland): +358 45 640 2013
skype: alexlittleou
twitter: @digicampusteam <https://twitter.com/digicampusteam> &
@alexlittle <https://twitter.com/alexlittle>
|
The immediate issue is that second time you ran a different Make command, @courcelan is going to update in the PR (which was missing so you wouldn't have seen any updates anyhow!) but the reason for updating the Vue deps has to do with development mode synchronization. All of the application JS and assets will have otherwise been compiled already. |
As mentioned on basecamp - this may be because I was using the browser back/forward buttons
The text was updated successfully, but these errors were encountered: