Skip to content

View and router frontend CS#231

Merged
chrisdavenport merged 10 commits intojoomla-extensions:masterfrom
zero-24:patch-3
Jul 3, 2016
Merged

View and router frontend CS#231
chrisdavenport merged 10 commits intojoomla-extensions:masterfrom
zero-24:patch-3

Conversation

@zero-24
Copy link
Copy Markdown
Contributor

@zero-24 zero-24 commented Jun 28, 2016

Pull Request for Issues at: https://travis-ci.org/yvesh/weblinks/jobs/140624371

Summary of Changes

CS

Testing Instructions

please review the code as it just CS fixes. If you want you can test the frontend of com_weblinks

*
* @return array The URL arguments to use to assemble the subsequent URL.
*
* Note. These functions are proxys for the new router interface
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"proxys" should be "proxies".

@zero-24
Copy link
Copy Markdown
Contributor Author

zero-24 commented Jun 29, 2016

fixed @chrisdavenport

@chrisdavenport chrisdavenport merged commit 77d2b1f into joomla-extensions:master Jul 3, 2016
@zero-24 zero-24 deleted the patch-3 branch July 3, 2016 13:35
chrisdavenport pushed a commit that referenced this pull request Jul 4, 2016
* allow captcha  on submit web link

allow captcha  on submit web link (1/5)

* allow captcha  on submit web link

allow captcha  on submit web link (2/5)

* allow captcha  on submit web link

allow captcha  on submit web link (3/5)

* allow captcha  on submit web link

allow captcha  on submit web link (4/5)

* allow captcha  on submit web link

allow captcha  on submit web link (5/5)

* CS fix

CS fix

* XML CS for #223

Simple XML CS #223

* update captcha message

update captcha message like com_user, com_contatct #10931

* handle session data

handle session data 1(/2)

* handle session data

handle session data (2/2)

* move buttons down

move buttons down

* fix conflicts with #231

fix conflicts with #231

* fix conflicts with #231 

fix conflicts with #231
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.

2 participants