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

Apply fixes from StyleCI #1614

Merged
merged 1 commit into from
Jan 3, 2017
Merged

Apply fixes from StyleCI #1614

merged 1 commit into from
Jan 3, 2017

Conversation

DawoudIO
Copy link
Contributor

@DawoudIO DawoudIO commented Jan 3, 2017

This pull request applies code style fixes from an analysis carried out by StyleCI.


For more information, click here.

@DawoudIO DawoudIO added this to the 2.5.0 milestone Jan 3, 2017
@DawoudIO DawoudIO merged commit 81e9c8e into StyleCI-PSR2 Jan 3, 2017
@DawoudIO DawoudIO deleted the analysis-zYDDmY branch January 3, 2017 04:22
@DawoudIO DawoudIO removed the In Review label Jan 3, 2017
DawoudIO added a commit that referenced this pull request Jan 3, 2017
* added master file

* fixed echo issue

* Apply fixes from StyleCI (#1614)
DawoudIO added a commit that referenced this pull request Jan 9, 2017
* starting 2.5.0

* added upgrade.json hint

* New Wercker with PHP/Composer/Node

* reformat script

* v3

* back to simple build.

* break down into smaller scripts

* composer version echo

* Some gettext more

Some gettext more

* Changed PHP tag

Changed PHP tag

* Fix #1552 (#1559)

Fix #1552

* Delete wercker.yml

* Moving to a ChurchCRM Box that has all the system level changes

* Moved checkAllowedURL to SystemURLs

- Checking Once at LocalConfig Load
- Tested setting BlockURL to true and redirect

* move box to https

* start mailcatcher with vagrant (#1580)

* moved to bootstrap.sh

* moved ruby / gem install here

* gen files should not be checked in

* removed auto gen files

* added grunt sass to gen new files...

* part of the box setup now

* removed dbl imported code via new churchcrm.css / scss

* Apply fixes from StyleCI (#1610)

* Apply fixes from StyleCI (#1611)

* Style ci psr2 (#1615)

* added master file

* fixed echo issue

* Apply fixes from StyleCI (#1614)

* Update README.md

* Family External Information Verify (#1570)

* 1st commit for verify code

working views
- enter data
- view family

* add get people by Role

* fixed get photo

* using SystemURLs::getDocumentRoot()

* added getPeopleSorted

* sRootPath = SystemURLs::getRootPath()

* no longer passing sRootPath

* people are displayed in order

* added Tokens table

* removed invalid head Classification

* added missing SystemURLs use

* fixed ORM debug string

* verify by token is working

* added valid token check

* Renamed the class token not tokens

* final tokens table sql

* check to make sure we have the family in the db before rendering

* fixed renaming

* added more helper methdos and new isValid Logic

* added verify api to create the token for now

* php cleanup

* added getId

* moved FamilyVerify to API call

* Moved family verify to API call

* moved verifyDownloadPDF to button click and remove Email PDF

* there is no role 7

* Send Email with Verification Token

* Moved Confirm PDF email to Email Classes

* Moved ConfirmReport to Member Dashbaord

No need to hide it

* removed unsued buttons

* created BaseFamilyVerification

* added default index to getURL

* MVP no self request of the info

* moved JS to new JS file

* passing token to page

* new flooting buttons for verify/update/remove info

* now consume a use if a token has a max count

* Update Token Table

- renamed useCount to setRemainingUses
- updated verify to verifyFamily
- added verifyPerson

* renamed emails var to toAddresses

* updated route to /families/{id}/family

* default auth on smtp is off

* fixed naming issue

* Single verify button

* added gettext where missing

* added 404 page

* check group size

* select verification

* adding note when user submits info

* submit data and handle errors

* fixed getSendNewsletter

* fixed SMTP

* delete old token for the fmaily

* format cleanup

* Updated Confirm button with text also

* Apply fixes from StyleCI (#1619)

* develop mergeadded ruby

* fixed apt-get

* sudo apt-get install ruby-full

* sudo gem install sass
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.

1 participant