feat: added support for both legacy and new design (#349)

src/data/utils/cookies.js

@@ -1,21 +0,0 @@
-import Cookies from 'universal-cookie';
-import { getConfig } from '@edx/frontend-platform';
-
-export function setCookie(cookieName, cookieValue, cookieExpiry) {
-  const cookies = new Cookies();
-  const options = { domain: getConfig().COOKIE_DOMAIN, path: '/' };
-  if (cookieExpiry) {
-    options.expires = cookieExpiry;
-  }
-  cookies.set(cookieName, cookieValue, options);
-}
-
-export default function setSurveyCookie(surveyType) {
-  const cookieName = getConfig().USER_SURVEY_COOKIE_NAME;
-  if (cookieName) {
-    const signupTimestamp = (new Date()).getTime();
-    // set expiry to exactly 24 hours from now
-    const cookieExpiry = new Date(signupTimestamp + 1 * 864e5);
-    setCookie(cookieName, surveyType, cookieExpiry);
-  }
-}

src/data/utils/dataUtils.js

@@ -1,80 +0,0 @@
-// Utility functions
-
-import * as QueryString from 'query-string';
-import { AUTH_PARAMS } from '../constants';
-
-export default function processLink(link) {
-  let matches;
-  link.replace(/(.*?)<a href=["']([^"']*).*?>([^<]+)<\/a>(.*)/g, function () { // eslint-disable-line func-names
-    matches = Array.prototype.slice.call(arguments, 1, 5); // eslint-disable-line  prefer-rest-params
-  });
-  return matches;
-}
-
-export const getTpaProvider = (tpaHintProvider, primaryProviders, secondaryProviders) => {
-  let tpaProvider = null;
-  let skipHintedLogin = false;
-  [...primaryProviders, ...secondaryProviders].forEach((provider) => {
-    if (provider.id === tpaHintProvider) {
-      tpaProvider = provider;
-      if (provider.skipHintedLogin) {
-        skipHintedLogin = true;
-      }
-    }
-  });
-  return { provider: tpaProvider, skipHintedLogin };
-};
-
-export const getTpaHint = () => {
-  const params = QueryString.parse(window.location.search);
-  let tpaHint = null;
-  tpaHint = params.tpa_hint;
-  if (!tpaHint) {
-    const { next } = params;
-    if (next) {
-      const index = next.indexOf('tpa_hint=');
-      if (index !== -1) {
-        tpaHint = next.substring(index + 'tpa_hint='.length, next.length);
-      }
-    }
-  }
-  return tpaHint;
-};
-
-export const updatePathWithQueryParams = (path) => {
-  const queryParams = window.location.search;
-
-  if (!queryParams) {
-    return path;
-  }
-
-  return `${path}${queryParams}`;
-};
-
-export const getAllPossibleQueryParam = () => {
-  const urlParams = QueryString.parse(window.location.search);
-  const params = {};
-  Object.entries(urlParams).forEach(([key, value]) => {
-    if (AUTH_PARAMS.indexOf(key) > -1) {
-      params[key] = value;
-    }
-  });
-
-  return params;
-};
-
-export const getActivationStatus = () => {
-  const params = QueryString.parse(window.location.search);
-
-  return params.account_activation_status;
-};
-
-export const isScrollBehaviorSupported = () => 'scrollBehavior' in document.documentElement.style;
-
-export const windowScrollTo = (options) => {
-  if (isScrollBehaviorSupported()) {
-    return window.scrollTo(options);
-  }
-
-  return window.scrollTo(options.top, options.left);
-};

src/data/utils/dataUtils.test.js

@@ -1,32 +0,0 @@
-import { LOGIN_PAGE } from '../constants';
-import processLink, { updatePathWithQueryParams } from './dataUtils';
-
-describe('processLink', () => {
-  it('should use the provided processLink function to', () => {
-    const expectedHref = 'http://test.server.com/';
-    const expectedText = 'test link';
-    const link = `<a href="${expectedHref}">${expectedText}</a>`;
-
-    const matches = processLink(link);
-
-    expect(matches[1]).toEqual(expectedHref);
-    expect(matches[2]).toEqual(expectedText);
-  });
-});
-
-describe('updatePathWithQueryParams', () => {
-  it('should append query params into the path', () => {
-    const params = '?course_id=testCourseId';
-    const expectedPath = `${LOGIN_PAGE}${params}`;
-
-    Object.defineProperty(window, 'location', {
-      value: {
-        href: 'http://localhost/',
-        search: params,
-      },
-    });
-    const updatedPath = updatePathWithQueryParams(LOGIN_PAGE);
-
-    expect(updatedPath).toEqual(expectedPath);
-  });
-});

src/data/utils/index.js

@@ -1,11 +0,0 @@
-export {
-  default,
-  getTpaProvider,
-  getTpaHint,
-  updatePathWithQueryParams,
-  getAllPossibleQueryParam,
-  getActivationStatus,
-  windowScrollTo,
-} from './dataUtils';
-export { default as AsyncActionType } from './reduxUtils';
-export { default as setSurveyCookie, setCookie } from './cookies';

src/data/utils/reduxUtils.js

@@ -1,34 +0,0 @@
-/**
- * Helper class to save time when writing out action types for asynchronous methods.  Also helps
- * ensure that actions are namespaced.
- */
-export default class AsyncActionType {
-  constructor(topic, name) {
-    this.topic = topic;
-    this.name = name;
-  }
-
-  get BASE() {
-    return `${this.topic}__${this.name}`;
-  }
-
-  get BEGIN() {
-    return `${this.topic}__${this.name}__BEGIN`;
-  }
-
-  get SUCCESS() {
-    return `${this.topic}__${this.name}__SUCCESS`;
-  }
-
-  get FAILURE() {
-    return `${this.topic}__${this.name}__FAILURE`;
-  }
-
-  get RESET() {
-    return `${this.topic}__${this.name}__RESET`;
-  }
-
-  get FORBIDDEN() {
-    return `${this.topic}__${this.name}__FORBIDDEN`;
-  }
-}

src/data/utils/reduxUtils.test.js

@@ -1,14 +0,0 @@
-import AsyncActionType from './reduxUtils';
-
-describe('AsyncActionType', () => {
-  it('should return well formatted action strings', () => {
-    const actionType = new AsyncActionType('HOUSE_CATS', 'START_THE_RACE');
-
-    expect(actionType.BASE).toBe('HOUSE_CATS__START_THE_RACE');
-    expect(actionType.BEGIN).toBe('HOUSE_CATS__START_THE_RACE__BEGIN');
-    expect(actionType.SUCCESS).toBe('HOUSE_CATS__START_THE_RACE__SUCCESS');
-    expect(actionType.FAILURE).toBe('HOUSE_CATS__START_THE_RACE__FAILURE');
-    expect(actionType.RESET).toBe('HOUSE_CATS__START_THE_RACE__RESET');
-    expect(actionType.FORBIDDEN).toBe('HOUSE_CATS__START_THE_RACE__FORBIDDEN');
-  });
-});