Skip to content
This repository was archived by the owner on Aug 31, 2019. It is now read-only.

Lm/toasted errors #100

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Lm/toasted errors #100

wants to merge 2 commits into from

Conversation

lukemalinowski
Copy link
Contributor

Add toasted error display for when JSON.parse fails.

Alternative is to make a Toast component, one does exist https://github.com/ktsn/vuex-toast, but it doesn't look super clean, and we can write our own properly namespaced one for our store in around 50 lines

Signed-off-by: Luke Malinowski <[email protected]>
Add callback option to dispatcher for load_file, only one so far,
that is for if JSON.parse() errors out due to syntax error.

A better way to do this is to write a Toast component, and send
TOAST_MESSAGES through vuex whenever an error happens, but this works
for testing now.

Related to #97

Signed-off-by: Luke Malinowski <[email protected]>
@lukemalinowski lukemalinowski added the enhancement New feature or request label Aug 6, 2019
@lukemalinowski lukemalinowski requested a review from Mitriol August 6, 2019 14:36
@lukemalinowski lukemalinowski self-assigned this Aug 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant