Skip to content

[MIG][10.0] website_form_recaptcha - #267

Merged
lasley merged 1 commit into
OCA:10.0from
LasLabs:release/10.0/website_form_recaptcha
Mar 28, 2017
Merged

lasley merged 1 commit into
OCA:10.0from
LasLabs:release/10.0/website_form_recaptcha

Conversation

@lasley

@lasley lasley commented Oct 19, 2016

Copy link
Copy Markdown
Contributor

This was theoretically as simple as changing the version numbers. I would like a functional test though, so WIP until my PR on website_crm_recaptcha so that we may confirm the base works.

  • Update website_form_recaptcha to v10

@lasley lasley added this to the 10.0 milestone Oct 19, 2016
@lasley lasley changed the title [MIG] website_form_recaptcha: Upgrade to v10 [MIG][10.0] website_form_recaptcha Oct 19, 2016
@pedrobaeza pedrobaeza mentioned this pull request Oct 19, 2016
35 tasks
@lasley lasley mentioned this pull request Oct 20, 2016
1 task
@lasley

lasley commented Oct 22, 2016

Copy link
Copy Markdown
Contributor Author

This has been fixed & is ready for review.

Functional review can take place in #268

@lasley

lasley commented Mar 17, 2017

Copy link
Copy Markdown
Contributor Author

@tedsalmon please review

@tedsalmon tedsalmon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍 LGTM!

@simahawk simahawk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM despite a small issue w/ JS indentation

url: '/website/recaptcha/',
method: 'GET',
success: function(data){
data = JSON.parse(data);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

bad indentation here ;)

* Update website_form_recaptcha to v10
@lasley
lasley force-pushed the release/10.0/website_form_recaptcha branch from 9c9a9ea to 5e7afaf Compare March 28, 2017 17:48
@lasley
lasley merged commit 65bb6c2 into OCA:10.0 Mar 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants