Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

sync blocker for auto toggle #107

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

Merged
merged 1 commit into from
Oct 19, 2019
Merged

sync blocker for auto toggle #107

merged 1 commit into from
Oct 19, 2019

Conversation

veebuv
Copy link
Contributor

@veebuv veebuv commented Mar 27, 2019

No description provided.

@veebuv veebuv force-pushed the remove-auto-sync branch from aa6e821 to de5a513 Compare March 27, 2019 01:31
@veebuv veebuv force-pushed the remove-auto-sync branch from de5a513 to e69afe9 Compare March 27, 2019 01:41
@veebuv
Copy link
Contributor Author

veebuv commented Mar 27, 2019

The value sent back on changed will still be the toggled value so the emit event can send that to the parent, the parent can act accordingly, make API calls etc, and once data is resolved, set the value to true, which will auto toggle

@ninhxuanluu
Copy link

need to review asap @euvl, pls

@simondotwhite
Copy link

Can this be merged @euvl? Thanks

src/Button.vue Show resolved Hide resolved
@euvl euvl merged commit 4dab16c into euvl:master Oct 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.