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

remove scrollback.io and other cruft #96

Merged
merged 1 commit into from
Sep 11, 2014

Conversation

kaihendry
Copy link
Member

scrollback.io and davis is slowing down the site. This is especially apparent on the mobile site. This is a purge of it to make the site faster.

We have tried scrollback.io over the last year. I for one think it has brought zero utility. There are better Web logging applications, e.g. https://botbot.me/ There are better Web IRC clients https://news.ycombinator.com/item?id=8285832

google analytics was removed too since this is the older slower method and should be probably using its new Universal API or just use the server logs, e.g. with http://goaccess.io/

I also took the liberty of updating jquery. Though we should probably replace jquery with the even saner http://zeptojs.com/

@notthetup
Copy link
Member

👍 for removing scrollback. I agree it doesn't add much value.

👎 for removing google analytics without replacing with the new Universal Analytics. It isn't that hard. Server logs are great, but we head back to the issue of who's gonna maintain a server for goaccess etc.

👍 for zepto.

@rolandturner
Copy link
Contributor

Kai,

Why are you removing scrollback.io?

  • Roland

On 09/11/2014 02:41 PM, Kai Hendry wrote:


    You can merge this Pull Request by running

git pull https://github.com/kaihendry/hackerspace.sg noscrollback

Or view, comment on, or merge it at:

#96

    Commit Summary


Reply to this email directly or view it on GitHub
#96.

@rolandturner
Copy link
Contributor

Erm, why is Google Analytics being pulled?

  • Roland

On 09/11/2014 02:58 PM, Chinmay Pendharkar wrote:

👍 for removing scrollback. I agree it doesn't add much value.

👎 for removing google analytics without replacing with the new
Universal Analytics. It isn't that hard. Server logs are great, but we
head back to the issue of who's gonna maintain a server for goaccess etc.

👍 for zepto.


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

@kaihendry
Copy link
Member Author

As mentioned in the description, and as a longtime IRC user, it provides no utility @rolandturner

Read the description. I used hub to create the pull request and there seems to be no way to populate the description.

@robertsky
Copy link
Member

@rolandturner, @notthetup: @kaihendry doesn't have access to Google Analytics, but that doesn't give him an excuse to pull Google Analytics from the site. :P

@kaihendry, please update the Google Analytics code to this.

<script>
  (function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){
  (i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),
  m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)
  })(window,document,'script','//www.google-analytics.com/analytics.js','ga');

  ga('create', 'UA-11667249-1', 'auto');
  ga('send', 'pageview');

</script>

or if anyone is feeling adventurous - Google Tag Manager:

<!-- Google Tag Manager -->
<noscript><iframe src="//www.googletagmanager.com/ns.html?id=GTM-M3BQNB"
height="0" width="0" style="display:none;visibility:hidden"></iframe></noscript>
<script>(function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src=
'//www.googletagmanager.com/gtm.js?id='+i+dl;f.parentNode.insertBefore(j,f);
})(window,document,'script','dataLayer','GTM-M3BQNB');</script>
<!-- End Google Tag Manager -->

@valentine
Copy link
Member

Just added @kaihendry and @notthetup to Analytics.

@kaihendry
Copy link
Member Author

I've amended and put the GA universal snippet in there. We must submit to Google. 😺

@robertsky
Copy link
Member

Must. Have. Data. ℹ️

@chernjie
Copy link
Member

@kaihendry may I suggest you break down this commit to three different pull request? ga, davis, and scrollback.io

Smaller atomic pull request get merged faster

@kaihendry
Copy link
Member Author

@chernjie no, it will be faster if you merge it right now. These changes are trivial.

notthetup added a commit that referenced this pull request Sep 11, 2014
remove scrollback.io and other cruft
@notthetup notthetup merged commit a93d85a into hackerspacesg:master Sep 11, 2014
@rolandturner
Copy link
Contributor

On 09/11/2014 03:07 PM, Kai Hendry wrote:

As mentioned in the description, and as a longtime IRC users, it
provides no utility @rolandturner https://github.com/rolandturner

Apologies, the description didn't appear in the email (something is
presumably broken somewhere). So:

scrollback.io and davis is slowing down the site. This is especially
apparent on the mobile site. This is a purge of it to make the site
faster.

We have tried scrollback.io over the last year. I for one think it has
brought zero utility. There are better Web logging applications, e.g.
https://botbot.me/ There are better Web IRC clients
https://news.ycombinator.com/item?id=8285832

google analytics was removed too since this is the older slower method
and should be probably using its new Universal API or just use the
server logs, e.g. with http://goaccess.io/

I also took the liberty of updating jquery. Though we should probably
replace jquery with the even saner http://zeptojs.com/

If Scrollback is integrated correctly, it can't slow the site down. It's
not aimed at longtime IRC users (who presumably already have IRC clients
that they'd prefer to use) but at casual users who aren't interested in
running yet another client. That latter category includes me so, no, the
utility is not zero.

I note that the pull has happened, can we please undo this?

  • Roland

@kaihendry
Copy link
Member Author

@rolandturner it does slow the site down. Have a look at the paint refreshes in Google chrome dev tools and its esp bad on mobile.

If you need a Web client, (i.e. you don't want to install an IRC client locally I'll happily help setup a Web IRC client like shout. In the interim there is http://webchat.freenode.net/

@notthetup
Copy link
Member

I can add a data point as someone who's quite regularly in the IRC room.

I have yet to have a conversation with anyone who ended up on the #hackerspacesg IRC channel using scrollback.io. Most scrollback.io users usually just say "hello" and soon leave the room.

Point being it hasn't really added much interactivity to the website or hasn't gotten new regulars to our hackerspacesg channel.

@valentine
Copy link
Member

I noticed that the mailing list was cc'ed by @rolandturner. Goodness!

I agree with the removal of Scrollback widget. We can continue to link to Scrollback directly, but the widget should go!

@rolandturner
Copy link
Contributor

On 09/11/2014 09:25 PM, Chinmay Pendharkar wrote:

I can add a data point as someone who's quite regularly in the IRC
room. I have yet to have a conversation with anyone who ended up on
the #hackerspacesg IRC channel using scrollback.io. Most scrollback.io
users usually just say "hello" and soon leave the room.

That does point to Scrollback not being a good fit for HackerspaceSG.
The intention is specifically to allow non-IRC-users easy
(non-'90s-refugee-web-interface) access to IRC users within communities
for whom this this technology barrier is an issue. If we're not actually
having meaningful conversations happen this way then my occasional use
of it is perhaps insufficient to warrant retaining it.

Thanks.

  • Roland

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.

6 participants