Skip to content

Use returned value from toggle component on change event#205

Merged
liyasthomas merged 1 commit intohoppscotch:masterfrom
hosseinnedaee:code_refinment
Oct 22, 2019
Merged

Use returned value from toggle component on change event#205
liyasthomas merged 1 commit intohoppscotch:masterfrom
hosseinnedaee:code_refinment

Conversation

@hosseinnedaee
Copy link
Copy Markdown
Contributor

@hosseinnedaee hosseinnedaee commented Oct 22, 2019

A little code refinement
In this PR I used the actual returned value from the toggle component as $event to apply setting changes

@ghost
Copy link
Copy Markdown

ghost commented Oct 22, 2019

DeepCode Report (#cbbc0e)

DeepCode analyzed this pull request.
There are no new issues.

@TravisBuddy
Copy link
Copy Markdown

Hey @hosseinnedaee,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: fd3e5b20-f503-11e9-9e2b-01a46d152451

Copy link
Copy Markdown
Member

@liyasthomas liyasthomas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@liyasthomas liyasthomas added the feature New feature or request label Oct 22, 2019
@liyasthomas liyasthomas merged commit 7f7d509 into hoppscotch:master Oct 22, 2019
@hosseinnedaee hosseinnedaee deleted the code_refinment branch November 7, 2019 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants