Skip to content

Fix899#901

Merged
danbri merged 2 commits intomasterfrom
fix899
Nov 26, 2015
Merged

Fix899#901
danbri merged 2 commits intomasterfrom
fix899

Conversation

@RichardWallis
Copy link
Contributor

Fix to #899 - Fix to ShortenOnSentence() which failed on comment string over max length which did not end with '.'. For completeness, added the missing '.' to definition of vehicleSeatingCapacity property.

Also Disabled _siteDebug call on host schema.org unless PRODSITEDEBUG environment variable set to 'True'. Still operates as before on all other hosts.

RichardWallis added 2 commits November 19, 2015 14:24
…ironment variable set to 'True'. Sitll operates on all other hosts.
Fix to ShortenOnSentence() which failed on comment sting over max length which did not end with '.'.  For completenes, added the missing '.' to definition of vehicleSeatingCapacity property.
danbri added a commit that referenced this pull request Nov 26, 2015
@danbri danbri merged commit 1d4639a into master Nov 26, 2015
@danbri
Copy link
Contributor

danbri commented Nov 26, 2015

Ok let's get this released 'early access'...

@RichardWallis
Copy link
Contributor Author

Yup - you going to do that - and update deimos to bring it into step?

~Richard.

On Thu, Nov 26, 2015 at 6:46 PM, Dan Brickley [email protected]
wrote:

Ok let's get this released 'early access'...


Reply to this email directly or view it on GitHub
#901 (comment).

@danbri
Copy link
Contributor

danbri commented Nov 27, 2015

I will! Thanks for the fix.

On Thu, Nov 26, 2015 at 11:05 PM RichardWallis [email protected]
wrote:

Yup - you going to do that - and update deimos to bring it into step?

~Richard.

On Thu, Nov 26, 2015 at 6:46 PM, Dan Brickley [email protected]
wrote:

Ok let's get this released 'early access'...


Reply to this email directly or view it on GitHub
<#901 (comment)
.


Reply to this email directly or view it on GitHub
#901 (comment).

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.

3 participants