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

feat: Added analytics logging on non-admin pages access #229

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
18 changes: 13 additions & 5 deletions lib/firebase-client.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import firebase from 'firebase/app';
import 'firebase/analytics';

export const firebaseConfig = {
apiKey: process.env.NEXT_PUBLIC_FIREBASE_API_KEY,
Expand All @@ -15,10 +16,17 @@ export const firebaseConfig = {
*
* See the contributor docs for more info.
*/
export function initFirebase() {
if (firebase.apps.length > 0) {
return;
}

if (!firebase.apps.length) {
firebase.initializeApp(firebaseConfig);
}

const analytics = firebase.analytics;

export { firebase, analytics };
// export function initFirebase() {
// if (firebase.apps.length > 0) {
// return;
// }

// firebase.initializeApp(firebaseConfig);
// }
22 changes: 19 additions & 3 deletions pages/_app.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import Head from 'next/head';
import { AppProps } from 'next/dist/shared/lib/router/router';
import 'firebase/auth';
import AppHeader from '../components/AppHeader';
import { initFirebase } from '../lib/firebase-client';
import { analytics } from '../lib/firebase-client';
import { AuthProvider } from '../lib/user/AuthContext';
import '../styles/globals.css';
import '../styles/tailwind.css';
Expand All @@ -18,8 +18,8 @@ import 'react-notion-x/src/styles.css';
import 'prismjs/themes/prism-tomorrow.css';
// used for rendering equations
import 'katex/dist/katex.min.css';

initFirebase();
import { useEffect } from 'react';
import { useRouter } from 'next/router';

/**
* A Wrapper for the HackPortal web app.
Expand All @@ -28,6 +28,22 @@ initFirebase();
* will load into memory and never re-initialize unless the page refreshes.
*/
function PortalApp({ Component, pageProps }: AppProps) {
const router = useRouter();

useEffect(() => {
if (process.env.NODE_ENV === 'production') {
const logEvent = (url: string) => {
analytics().setCurrentScreen(url);
analytics().logEvent('screen_view');
};
router.events.on('routeChangeComplete', logEvent);
logEvent(window.location.pathname);
return () => {
router.events.off('routeChangeComplete', logEvent);
};
}
});

return (
<DndProvider backend={HTML5Backend}>
<LocalizationProvider dateAdapter={AdapterDateFns}>
Expand Down
1 change: 0 additions & 1 deletion pages/register.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ import LoadIcon from '../components/LoadIcon';
import { useUser } from '../lib/profile/user-data';
import { RequestHelper } from '../lib/request-helper';
import { useAuthContext } from '../lib/user/AuthContext';
import firebase from 'firebase/app';
import { Formik, Form, Field, ErrorMessage } from 'formik';
import schools from '../public/schools.json';
import majors from '../public/majors.json';
Expand Down
1 change: 0 additions & 1 deletion pages/schedule/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ import PinDrop from '@material-ui/icons/PinDrop';
import ClockIcon from '@material-ui/icons/AccessTime';
import Backpack from '@material-ui/icons/LocalMall';
import Description from '@material-ui/icons/BorderColor';
import firebase from 'firebase';

const styles = ({ palette }: Theme) =>
createStyles({
Expand Down