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

Conversation

dsmewara
Copy link

Summary of Changes

Created test case scenarios for banners.

@puneet0191
Copy link
Member

Thanks, Good Work here 👍

I need to create modify trash archived Check-In And publish and Unpublish Banner

Background:
Given Joomla CMS is installed
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for the pull @dsmewara ^_^

This given needs to be removed as agreed at #54 (comment).

dsmewara added 3 commits July 26, 2016 11:22
Then I should see the "1 banner successfully deleted." message



No newline at end of file
Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove extra space as there should be only one empty line at end of the file.

@yvesh
Copy link
Contributor

yvesh commented Aug 9, 2016

@dsmewara can you please look at the suggestions from Prital and Puneet? Also your PR has merge conflicts now. Thank you for your work 👍

@Kubik-Rubik
Copy link
Contributor

Thank you @dsmewara! We remade your PR at the Automated Testing Code Sprint: #94

yvesh pushed a commit that referenced this pull request Jun 17, 2017
* Codestyle

* indent

* order

* oops

* changes requested by @andrepereiradasilva

* cs

* changes requested by @wojsmol

* Update mod_logged.xml (#43)

* Update templateDetails.xml (#45)

i need to fix my ide!!

* Update mod_popular.xml (#44)

* Update templateDetails.xml (#57)

* Update mod_version.xml (#56)

* Update mod_toolbar.xml (#55)

* Update mod_title.xml (#54)

* Update mod_submenu.xml (#53)

* Update mod_status.xml (#52)

* Update mod_stats_admin.xml (#51)

* Update mod_quickicon.xml (#50)

* Update mod_menu.xml (#49)

* Update mod_login.xml (#48)

* Update mod_latest.xml (#47)

* Update mod_feed.xml (#46)
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.

6 participants