[WIP] Feathers vuex #228
Closed
[WIP] Feathers vuex #228
+1,457
−1,442
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
roschaefer
commented
Aug 3, 2018
store/index.js
Outdated
| } | ||
|
|
||
| export const plugins = [ | ||
| service('usersettings') |
Contributor
Author
There was a problem hiding this comment.
calling this.$store.dispatch('usersettings/find', {}) will run a number of mutations and switches isFindPending automagically: https://feathers-plus.github.io/v1/feathers-vuex/service-module.html
roschaefer
force-pushed
the
feathers-vuex
branch
2 times, most recently
from
478b42b to
f233301
Compare
- no bash scripts * error prone * platform dependent)
* .nuxt seems to be a build folder and should not be synced * load config into process.env for local development
... and remove redundancy in docker-compose files
With server side rendering it is simply not possible to run the webapp without a bridged network
The point of this docker-compose file is to give a running *local* environment. It is not complete up to this point. Eventually we could share this configuration with our staging system online. Also this commit passes secret environment variables from host system to container.
* Install nuxt-env * Replace env with app.$env when possible * Remove custom plugins/env.js
* add yarn run build to Dockerfile * remove envsub (now unnecessary) - all configurations are controlled via env variables passed from host to container * remove .env.template
roschaefer
force-pushed
the
feathers-vuex
branch
2 times, most recently
from
70d639e to
fe24fe3
Compare
Merged
Contributor
Author
|
dropped in favour of #292 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove obsoletefeathers-client.js