Skip to content
This repository was archived by the owner on Mar 17, 2020. It is now read-only.

Conversation

pritalpatel
Copy link
Contributor

@pritalpatel pritalpatel commented Jun 29, 2016

Pull Request for Issue #36 .

every page object have the $url property

@yvesh
Copy link
Contributor

yvesh commented Jun 30, 2016

If every page has a $url property, you could make an interface / abstract class requiring it. (Ping me if i should explain that further) You already went into that direction with your AdminPage.

@javigomez
Copy link
Contributor

I'm merging current pull request. I have created this issue for Yves recommendation #43

@javigomez javigomez merged commit 34aaf4d into joomla-projects:staging Jun 30, 2016
pritalpatel pushed a commit to pritalpatel/gsoc16_browser-automated-tests that referenced this pull request Jul 7, 2016
@pritalpatel pritalpatel deleted the fix_issue_#36 branch October 14, 2016 15:06
yvesh pushed a commit that referenced this pull request Jun 17, 2017
* codestyle

* code style

* codestyle

* codestyle

* codestyle

* thanks @wojsmol

* corrections - thanks @Quy

* corrections - thanks @Quy

* oops

* make @Quy happy

* Update article.xml

* Remove space

* Update config.xml (#14)

* Update filter.xml (#15)

* Update config.xml (#16)

* Update profile.xml (#17)

* Update application.xml (#18)

* Update article.xml (#19)

* Update filter_articles.xml (#20)

* Update config.xml (#24)

* Update config.xml (#23)

* Update filter_fields.xml (#22)

* Update filter_featured.xml (#21)

* Update override.xml (#25)

* Update config.xml

* Update config.xml (#26)

* Update itemadmin_alias.xml (#30)

* Update itemadmin.xml (#29)

* Update item.xml (#27)

* Update item_alias.xml (#28)

* Update itemadmin_url.xml (#31)

* Update module.xml (#32)

* Update plugin.xml (#33)

* Update config.xml (#34)

* Update link.xml (#35)

* Update config.xml (#36)

* Update style.xml (#38)

* Update config.xml (#37)

* Update note.xml (#42)

* Update group.xml (#41)

* Update filter_debuggroup.xml (#40)

* Update config.xml (#39)

* corrections for @andrepereiradasilva

* gotya
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants