Skip to content
This repository has been archived by the owner on Sep 25, 2019. It is now read-only.

Move to Firebase hosting. Remove server code. #102

Merged
merged 26 commits into from
Aug 28, 2016
Merged

Conversation

Splaktar
Copy link
Contributor

@Splaktar Splaktar commented Aug 15, 2016

  • UrlShortener and analytics server features removed.
  • Configure for Firebase Hosting and HTTPS.
  • Change routing to use gdg.events/android/events instead of android.gdg.events.
  • Allow location API to work in Chrome 50+ once again.

Please check it out here: https://firefly-1c312.firebaseapp.com/ and https://gdg.events/

Fixes #26, #58, #97, #80, #81.
Closes #82, #101, #96.

friedger and others added 16 commits December 2, 2015 10:37
…epeat`

Fix some styling issues (more to do still)
Refactor routing, components, and asset locations.
Remove empty files.
Added in some one time bindings for performance.
Filter in controller as much as possible for performance.
Use minified version of ngGeolocation.
Bump version of ngGeolocation.
Sometimes this was being stored in the $scope which wasn't always visible and caused duplication of data.
Update to Angular Material 1.0.0.
update dependencies
fix grunt copy task to copy image assets to `dist/` properly
@Splaktar Splaktar added this to the 1.3.2 milestone Aug 15, 2016
@Splaktar Splaktar mentioned this pull request Aug 15, 2016
Remove old dependencies from server.

Fixes #55.
fix JSCS failures
remove excess server requests and more code that assumes that the URL will be prefixed
add BrowserSync config for use with lite-server
add `local.env.sample.js` with instructions for configuring keys locally
remove UrlShortener docs
update local dev and production deployment steps
update License years
<md-list-item class="md-2-line" ng-repeat="event in allEvents | orderBy:distanceFromHere | limitTo:5"
ng-click="openEvent(event._id)">
<md-list-item class="md-2-line" ng-repeat="event in vm.nearEvents | limitTo:5"
ng-click="vm.openEvent(event._id)">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noob question. What does vm stand for?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ViewModel or Controller. Angular 1.x style guide uses vm and Angular 2 style guide has switched to $ctrl.

@tasomaniac
Copy link
Member

I wonder what do we think about the subdomain? Does Firebase provide ssl for subdomains? If so, we can keep the old url structure right?

@Splaktar
Copy link
Contributor Author

@tasomaniac I don't think that Firebase supports HTTPS for subdomains unless you want to deploy a separate Firebase for each subdomain. That would be a nightmare to maintain though.

@Splaktar
Copy link
Contributor Author

I'm seeing some funky behavior on the md-virtual-repeats on the specific tag pages like: https://gdg.events/devfest/events

The first 3 elements are repeating at the end and the sorting doesn't seem right. I'll try to look into it later.

@tasomaniac
Copy link
Member

Looks good to me. You can merge it once you fixed the problem.

@Splaktar
Copy link
Contributor Author

OK, fixed the odd behavior of repeating elements in md-virtual-repeat.

Also fixed

  • Clicking on the link in map markers didn't work
  • Tagged event page was loading events from all tags instead of just it's own tag
  • Updated tests and readme

@Splaktar
Copy link
Contributor Author

The only remaining issue that I'm seeing is related to the Hub. Sometimes I'm not getting back all of the results that I would expect querying for a tag like DevFest for upcoming events. I'm investigating that separate from this PR.

@tasomaniac
Copy link
Member

👍 Great work.

@Splaktar
Copy link
Contributor Author

This looks like it will save about $70 per month on GCP hosting costs. Perhaps more when counting static IP and load balancer, etc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants