neojato / DeLorean-v2

A Professional Template for large GDG events. This project has been designed to continue to be your DevFest/IWD website solution without having to create a new site or Firebase project every year.
https://delorean-demo.firebaseapp.com
MIT License
47 stars 54 forks source link

Optimise usage of async pipe #17

Closed beeman closed 6 years ago

beeman commented 6 years ago

Description

This PR refactors some of the templates to optimise the usage of the async pipe.

Related Issue

https://github.com/neojato/DeLorean-v2/issues/16

Motivation and Context

I was going through the code and I realised that there are templates that subscribe to the same observable multiple times, so I patched it to optimise it.

How Has This Been Tested?

I manually tested the changes in the pages as I made them.

Screenshots (if appropriate):

Output should be the same.

These days it's becoming popular to suffix observables in the components with a $, like speaker$. I applied this to the HomeComponent, probably makes sense to apply it throughout the components.

Please let me know if you like to see anything changed in this PR :)