Jump to content

LaCollision

Clients
  • Posts

    1,001
  • Joined

  • Last visited

  • Days Won

    3

 Content Type 

Downloads

Release Notes

IPS4 Guides

IPS4 Developer Documentation

Invision Community Blog

Development Blog

Deprecation Tracker

Providers Directory

Forums

Events

Store

Gallery

Posts posted by LaCollision

  1. Hi Invision,

    Something is bothering me with the new referral system on IPS 4.5.

    In /Session/Front:

    public function init()
    {
    	/* Get cached page if available */
    	$this->checkCached();
    
    
    	[…]
    
    	/* Set a referral cookie */
    	if( \IPS\Settings::i()->ref_on and isset( \IPS\Request::i()->_rid ) )
    	{
    		\IPS\Request::i()->setCookie( 'referred_by', \intval( \IPS\Request::i()->_rid ), \IPS\DateTime::create()->add( new \DateInterval( 'P1Y' ) ) );
    	}
    
    	[…]
    }

     

    Doesn't it look like that when a page is cached, the referral system doesn't work, as the page is sent before a cookie is set?

     

    Thank you,

  2. Thank you!

    Yes I know this is an intended change – however many people also don't want to give their card number for a free trial, and that could stop them from buying the product right from the beginning – like: « Oh no, I'm not going to give again my credit card number just for trying a new product. » 🙂 

     

    2 minutes ago, bfarber said:

    Either way, I'm not sure this is really an ideal topic for the development gateway forum here. Would you like me to move this to the feedback forum instead?

    Yes, why not!

    However, could you give me just a few hints on where to look to change the code, in order not to require the credit card for a trial period?

    Thanks again!

  3. Hi everyone!

    Prior to version 4.5, purchases did not require a credit card number when the invoice amount was zero (e.g. for trial periods).
    This was a very useful feature to increase customer engagement, for whom giving out their credit card was often a hindrance.

    Since version 4.5, the credit card number is unfortunately required for all purchases with renewal, even those with a free trial period.

    I'll try to look in the code to make some changes, but it doesn't look simple to me.

    Do you have an idea on how to simply restore the behavior of Commerce 4.4?

    Many thanks! 🤗

  4. Hi IPS,

    Sorry if that question has already been asked a multizillion times – but I couldn't find it.

    Is the integration of CKEditor 5 something that you have planned?

    I know it's a big deal, but aren't there some improvements the Suite could use? Or maybe simply benefit from issues correction, as the v4 always gives me some troubles (I often can't move the cursor, for instance, even in the latest IPS).

    As you're upgrading the UX in IPS 4.5, modernising the interface could be great. The text editor is maybe the most important thing in the Suite – it's the central gateway with which users interact with everything.

    Thank you!

  5. On 3/30/2020 at 4:10 PM, bfarber said:

    No, there is no way to go back to the previous method.

    We ran into an issue in our own code using HTML with this new method, which we've since addressed. It seems unlikely we will be returning to the older method for the MR output.

     

    Thank you for your answer!

  6. Hi Invision,

    Recently, the MultipleRedirect javascript (and CSS) has been modified: it now uses the CSS "content" property to display the MultipleRedirect progress label.

    However, it's a bit embarrassing, as we now can't display HTML in this progress label. The former method was simply using javascript .html() jQuery method to display this label, and we were able to use HTML.

    => Why such a change? Isn't it possible to go back to the previous method?

    Thank you!

  7. Hi Invision,

    Recently, the MultipleRedirect javascript (and CSS) has been modified: it now uses the CSS "content" property to display the MultipleRedirect progress label.

    However, it's a bit embarrassing, as we now can't display HTML in this progress label. The former method was simply using javascript .html() jQuery method to display this label, and we were able to use HTML.

    => Why such a change? Isn't it possible to go back to the previous method?

    Thank you!

  8. On 3/10/2019 at 6:39 PM, Daniel F said:

    We've improved this for a future release 🙂

    I'm not going to mention any version yet😇, please pay attention to the dev change topics , but in the future you'll be able to use a generic template for this.

    
    {template="richText" group="global" app="core" location="global" params="the content which includes the media "}
    		

     Template parameters:

    
    $value, $extraClasses=array(), $extraControllers=array(), $extraAttributes=array()

     

    and the template will include all the necessary code. This is also going to make it future proof, so if anything else changes, we'll only need to adjust this template 🙂

     

    Hi @Daniel F,

    I can see the richText template in 4.4.2, that's great, thank you!

    Just a suggestion:

    For SEO purposes, the rich text could be embed in a h1, or h2, or hx tag…

    So that would be great for the richText template to have an additional 'tag' parameter:

    <ips:template parameters="$value, $tag = 'div', $extraClasses=array(), $extraControllers=array(), $extraAttributes=array()" />
    {{$controllers = array_merge( array('core.front.core.lightboxedImages'), $extraControllers );}}
    <{$tag} class='ipsType_richText {expression="implode(' ', $extraClasses)"}' data-controller='{expression="implode(',', $controllers)"}' {{foreach $extraAttributes as $attribute}} {$attribute|raw}{{endforeach}}>
    	{$value|raw}
    </{$tag}>

     

    Thanks!

  9. If you develop an app / plugin, and that you want to display a WYSIWYG editor, you must add the controller:

    data-controller='core.front.core.lightboxedImages'

    … to the div that includes your WYSIWYG editor content.

    For instance:

    <div class="ipsType_richText ipsContained" data-controller="core.front.core.lightboxedImages">
      {$object->content|raw}
    </div>

     

  10. Hi,

    It looks like that the breadcrumb JSON script is currently empty?

    I can't see it on my community (4.2.3), neither on this website, under the list of javascript files:

    <script type='text/javascript'>
    		
    			ips.setSetting( 'date_format', jQuery.parseJSON('"mm\/dd\/yy"') );
    		
    			ips.setSetting( 'date_first_day', jQuery.parseJSON('0') );
    		
    			ips.setSetting( 'remote_image_proxy', jQuery.parseJSON('1') );
    		
    			ips.setSetting( 'ipb_url_filter_option', jQuery.parseJSON('"none"') );
    		
    			ips.setSetting( 'url_filter_any_action', jQuery.parseJSON('"allow"') );
    		
    			ips.setSetting( 'bypass_profanity', jQuery.parseJSON('0') );
    		
    		
    	</script>
    
    
    ### JSON breadcrumb structured data missing here?
    
    		
    
    		<script src="//use.typekit.net/aeh4hzr.js"></script>

    Thanks :) 

×
×
  • Create New...