Skip to content
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

Fixed bug with submitBtn selector #28

Merged
merged 1 commit into from
Jan 12, 2016
Merged

Fixed bug with submitBtn selector #28

merged 1 commit into from
Jan 12, 2016

Conversation

k-zakhariy
Copy link
Contributor

No description provided.

@k-zakhariy
Copy link
Contributor Author

Please merge pull-request

@madhur
Copy link
Contributor

madhur commented Jan 12, 2016

Can you describe what is the purpose of this pull request ?

@k-zakhariy
Copy link
Contributor Author

We use your library in our project, first of all - thanks a lot for it. And
we found a bug in code, there are one missing variable submitBtn that
throws an error;

2016-01-12 14:54 GMT+02:00 Madhur Ahuja notifications@github.com:

Can you describe what is the purpose of this pull request ?


Reply to this email directly or view it on GitHub
#28 (comment)
.

З повагою,

Захарій

madhur added a commit that referenced this pull request Jan 12, 2016
Fixed bug with submitBtn selector
@madhur madhur merged commit 1f0ddfd into Coraza:master Jan 12, 2016
@madhur
Copy link
Contributor

madhur commented Jan 12, 2016

👍

@k-zakhariy
Copy link
Contributor Author

Thanks! 😊

@k-zakhariy
Copy link
Contributor Author

When i try to pull new version , it pulls an old version, any ideas about this trouble ?
I tried with --force flag, and tried run 'bower cache clean' before, but no success.
When i download zip archive directly from github then version is updated, but i need to use bower
image

@madhur
Copy link
Contributor

madhur commented Jan 13, 2016

I am not aware of bower much. I think version needs to be incremented.

On Wed, Jan 13, 2016 at 2:46 PM, Zakhariy notifications@github.com wrote:

When i try to pull new version , it pulls an old version, any ideas about
this trouble ?
I tried with --force flag, and tried run 'bower cache clean' before, but
no success
[image: image]
https://cloud.githubusercontent.com/assets/5267851/12289877/dd176e1a-b9e6-11e5-8be6-ebd615ce4c3f.png


Reply to this email directly or view it on GitHub
#28 (comment)
.

Madhur
Senior Developer
9999 55 9848

@sheyooo
Copy link

sheyooo commented Jan 28, 2016

this will be fixed in the latest pull request

@k-zakhariy
Copy link
Contributor Author

How i can pull last changes ? When i run bower update angular-emoji-popup -f --save i have an old code, and bug that i fixed is present .

@sheyooo
Copy link

sheyooo commented Feb 1, 2016

@k-zakhariy the change is in the src/ folder the changes need to be compiled into the dist/ folder which i have done in a fork of my own and provided a pull request but the problem is the dependencies,, i used gulp for my compilation he uses grunt so he doesnt want to merge the one with gulp so you pull my fork or see it you can use grunt to compile the changes and ask for a merge..
If you go to the src/ folder you will find the fix there...

@k-zakhariy
Copy link
Contributor Author

but here i dont see any changes in src/, and i build all this files with gulp already from src/
screenshot

@sheyooo
Copy link

sheyooo commented Feb 1, 2016

@k-zakhariy bower install https://github.com/sheyooo/angular-emoji-popup.git thats my fork and its all fixed in my fork

@k-zakhariy
Copy link
Contributor Author

I tried, it's also don't work
`d:\OpenServer\domains\test>bower cache clean
bower deleted Cached package angular-emoji-popup: C:\Users\Admin\AppData\Local\bower\cache\packages\81e229f177f92993010bbcc631f03865\1.1.0

d:\OpenServer\domains\test>bower install https://github.com/sheyooo/angular-emoji-popup.git --save
bower angular-emoji-popup#* not-cached https://github.com/sheyooo/angular-emoji-popup.git#*
bower angular-emoji-popup#* resolve https://github.com/sheyooo/angular-emoji-popup.git#*
bower angular-emoji-popup#* checkout 1.1.0
bower angular-emoji-popup#* mismatch Version declared in the json (0.0.0) is different than the resolved one (1.1.0)
bower angular-emoji-popup#* invalid-meta angular-emoji-popup is missing "ignore" entry in bower.json
bower angular-emoji-popup#* resolved https://github.com/sheyooo/angular-emoji-popup.git#1.1.0

d:\OpenServer\domains\test>`

after bower install i have an old version. What i am doing wrong ?

@sheyooo
Copy link

sheyooo commented Feb 2, 2016

@k-zakhariy https://github.com/sheyooo/angular-emoji-popup thats my fork you can pull it directly with git or download the zip until madhur fixes it in his own repo,, he said he is busy

@madhur
Copy link
Contributor

madhur commented Feb 2, 2016

Will resolve all issues this week

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.

None yet

3 participants