Redirect / for non-logged in users?

Yuun · 2016-01-20T14:30:17.730Z

And it works quite nicely! Thanks @riking and @JSey for your little plugin that I’ve been using up until now.

glynhudson · 2016-04-21T12:22:02.366Z

Now that his patch has been integrated how can I implement on my site? I would like to direct logo click to /latest and make categories the first landing page to help new users.

Thanks

Yuun · 2016-04-21T13:25:17.921Z

Just copy and paste riking’s script here into </head> in Admin → Customize → CSS/HTML to have the logo redirect to /latest.

riking:

<script>
  var HomeLogoComponent = require('discourse/components/home-logo').default;
  HomeLogoComponent.reopen({
   targetUrl: function() {
       return '/latest';
   }.property(),   
  });
</script>

To make categories the landing page, go to Admin → Site Settings → Basic Setup and make categories the first entry in the list for the top menu setting.

glynhudson · 2016-04-21T14:23:09.000Z

Works a treat

Thanks so much for quick reply.

My new forum setup is coming along nicely
http://community.openenergymonitor.org. So much nicer than our old Durpal 6
PHP setup http://openenergymonitor.org/emon/forum

JSey · 2016-04-26T20:16:09.310Z

Uh - and @Yuun’s code broke again:

Error: Could not find module discourse/components/home-logo

Where did that disappear to? Did a quick recursive grep, but came up with nothing?!

Falco · 2016-04-26T20:18:37.905Z

It’s broke because the new header.

Help us test the new header code! feature

I’ve just merged another client side performance improvement for Discourse. Our site header has been re-written using the widget code that gave us a huge performance increase within topics. The site header is present on every page and our benchmarks showed that it often took up 1/4 of the initial rendering time of a Discourse page. With the new code, it renders roughly 8x faster than before, which should be a significant improvement on android devices. Since this is a major re-write, the odds …

JSey · 2016-04-26T20:22:33.096Z

Yup, seen that, thanks. I guess I will x-post my question there since it belongs to that new addition.

eviltrout · 2016-04-27T17:19:02.300Z

I’ve added a new widget setting to customize that url in this commit:

https://github.com/discourse/discourse/commit/02f771e66ead3fb6e861bfa1fd41d8c65d10ab1d

Here’s an example of how to use it via a customization:

<script type="text/discourse-plugin" version="0.4">
  api.changeWidgetSetting('home-logo', 'href', '/new-url')
</script>

Just replace /new-url whatever you want the path to be and you should be good to go.

Falco · 2016-04-27T17:34:36.226Z

I want to change the behavior of the home logo only when users are at my home page (window.location === ‘example.com/forum/’).

Should passing a function as the third parameter work?

eviltrout · 2016-04-27T17:35:26.182Z

Hmmm, how did you do that before? I thought targetUrl was just a parameter?

JSey · 2016-04-27T17:51:51.582Z

Thanks a lot Robin - looks great. Only that the snippet you posted just gives me this error:

TypeError: Attempted to assign to readonly property.

or, on Chrome:

Uncaught TypeError: Cannot set property 'href' of undefined

…and makes the whole header disappear.

I guess I’m just missing something here?

eviltrout · 2016-04-27T17:55:55.246Z

Did you get the latest master first? I tried it locally and it worked fine.

JSey · 2016-04-27T17:56:57.905Z

That’s the first thing I did before trying it! Is there a special section where I have to put the script code?

eviltrout · 2016-04-27T18:01:46.995Z

Aha it seems the tests-passed branch failed to build due to a failing test. I’m fixing it now.

edit: fixed the test. Give it 15-20 mins and it should have a new version for you.

Falco · 2016-04-27T18:11:05.463Z

Sorry, this was done in a hurry

  window.goToPortal = function() { 
      if (window.location.href === 'https://example.com.br/forum/') { 
          window.location = 'https://example.com.br';
      }
  };

  var HeaderView = require('discourse/views/header').default

  HeaderView.reopen({
    didInsertElement : function(){
      this._super();
      Ember.run.scheduleOnce('afterRender', this, this.afterRenderEvent);
    },
    afterRenderEvent : function(){
      var logo = $('img#site-logo.logo-big').parent()[0];
    
      logo.addEventListener('click', window.goToPortal, false);
    }
  });

Can you help me @eviltrout ?

JSey · 2016-04-27T19:25:10.041Z

Like a charm. Thanks!

eviltrout · 2016-04-29T16:41:49.867Z

Hmmm that is a little complicated to establish in the new settings. Does adding that event work in the case where the url is not https://example.com.br/forum/ ? It seems the click handler would do nothing.

Falco · 2016-04-29T17:03:19.940Z

eviltrout:

It seems the click handler would do nothing.

Yes, when the user isn’t at the forum home, I want the normal behavior (scroll up / go back to home).

Only when we are already at the forum home, I want to go back to my Portal home.

eviltrout · 2016-04-29T17:28:06.491Z

It’s quite tricky to do what you’re asking, because once an event is handled from the logo it prevents default.

I’ve added the ability for the href setting to be a function. Whatever value you return will be the destination when the user clicks the logo:

https://github.com/discourse/discourse/commit/1ac6dd194d811d4bb5609562dc52176ce2bdc34f

Falco · 2016-05-09T23:10:32.129Z

Should this work?

<script type="text/discourse-plugin" version="0.4">
  api.changeWidgetSetting('home-logo', 'href', function() { 
          return 'https://example.com.br';
  });
</script>

EDIT:

Setting it to ‘/another/page’ when using Discourse under ‘/forum’ also breaks .