Compare commits
1 Commits
dependabot
...
feat--remo
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
fbca354df7 |
1
.env
1
.env
@@ -37,5 +37,4 @@ TWITTER_HASHTAG=''
|
|||||||
TWITTER_URL=''
|
TWITTER_URL=''
|
||||||
USER_INFO_COOKIE_NAME=''
|
USER_INFO_COOKIE_NAME=''
|
||||||
SESSION_COOKIE_DOMAIN=''
|
SESSION_COOKIE_DOMAIN=''
|
||||||
ENABLE_JUMPNAV='true'
|
|
||||||
ENABLE_NOTICES=''
|
ENABLE_NOTICES=''
|
||||||
|
|||||||
@@ -37,5 +37,4 @@ TWITTER_HASHTAG='myedxjourney'
|
|||||||
TWITTER_URL='https://twitter.com/edXOnline'
|
TWITTER_URL='https://twitter.com/edXOnline'
|
||||||
USER_INFO_COOKIE_NAME='edx-user-info'
|
USER_INFO_COOKIE_NAME='edx-user-info'
|
||||||
SESSION_COOKIE_DOMAIN='localhost'
|
SESSION_COOKIE_DOMAIN='localhost'
|
||||||
ENABLE_JUMPNAV='true'
|
|
||||||
ENABLE_NOTICES=''
|
ENABLE_NOTICES=''
|
||||||
|
|||||||
@@ -36,5 +36,4 @@ TERMS_OF_SERVICE_URL='https://www.edx.org/edx-terms-service'
|
|||||||
TWITTER_HASHTAG='myedxjourney'
|
TWITTER_HASHTAG='myedxjourney'
|
||||||
TWITTER_URL='https://twitter.com/edXOnline'
|
TWITTER_URL='https://twitter.com/edXOnline'
|
||||||
USER_INFO_COOKIE_NAME='edx-user-info'
|
USER_INFO_COOKIE_NAME='edx-user-info'
|
||||||
ENABLE_JUMPNAV='true'
|
|
||||||
ENABLE_NOTICES=''
|
ENABLE_NOTICES=''
|
||||||
|
|||||||
@@ -109,9 +109,3 @@ TWITTER_URL
|
|||||||
unless this is set. Optional.
|
unless this is set. Optional.
|
||||||
|
|
||||||
Example: https://twitter.com/edXOnline
|
Example: https://twitter.com/edXOnline
|
||||||
|
|
||||||
ENABLE_JUMPNAV
|
|
||||||
Enables the new Jump Navigation feature in the course breadcrumbs, defaulted to the string 'true'.
|
|
||||||
Disable to have simple hyperlinks for breadcrumbs. Setting it to any other value but 'true' ('false','I love flags', 'etc' would disable the Jumpnav).
|
|
||||||
This feature flag is slated to be removed as jumpnav becomes default. Follow the progress of this ticket here:
|
|
||||||
https://openedx.atlassian.net/browse/TNL-8678
|
|
||||||
|
|||||||
@@ -1,6 +1,5 @@
|
|||||||
import React, { useMemo } from 'react';
|
import React, { useMemo } from 'react';
|
||||||
import PropTypes from 'prop-types';
|
import PropTypes from 'prop-types';
|
||||||
import { getConfig } from '@edx/frontend-platform';
|
|
||||||
import { FormattedMessage } from '@edx/frontend-platform/i18n';
|
import { FormattedMessage } from '@edx/frontend-platform/i18n';
|
||||||
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
|
import { FontAwesomeIcon } from '@fortawesome/react-fontawesome';
|
||||||
import { faHome } from '@fortawesome/free-solid-svg-icons';
|
import { faHome } from '@fortawesome/free-solid-svg-icons';
|
||||||
@@ -28,7 +27,7 @@ function CourseBreadcrumb({
|
|||||||
whiteSpace: 'nowrap',
|
whiteSpace: 'nowrap',
|
||||||
}}
|
}}
|
||||||
>
|
>
|
||||||
{ getConfig().ENABLE_JUMPNAV !== 'true' || content.length < 2 || !isStaff
|
{content.length < 2 || !isStaff
|
||||||
? (
|
? (
|
||||||
<a className="text-primary-500" href={`/course/${courseId}/${defaultContent.id}`}>
|
<a className="text-primary-500" href={`/course/${courseId}/${defaultContent.id}`}>
|
||||||
{defaultContent.label}
|
{defaultContent.label}
|
||||||
|
|||||||
@@ -1,7 +1,6 @@
|
|||||||
import React from 'react';
|
import React from 'react';
|
||||||
import { screen, render } from '@testing-library/react';
|
import { screen, render } from '@testing-library/react';
|
||||||
import { getAuthenticatedUser } from '@edx/frontend-platform/auth';
|
import { getAuthenticatedUser } from '@edx/frontend-platform/auth';
|
||||||
import { getConfig } from '@edx/frontend-platform';
|
|
||||||
import { useModel, useModels } from '../../generic/model-store';
|
import { useModel, useModels } from '../../generic/model-store';
|
||||||
import CourseBreadcrumbs from './CourseBreadcrumbs';
|
import CourseBreadcrumbs from './CourseBreadcrumbs';
|
||||||
|
|
||||||
@@ -11,7 +10,6 @@ jest.mock('@edx/frontend-platform/analytics');
|
|||||||
// Remove When Fully rolled out>>>
|
// Remove When Fully rolled out>>>
|
||||||
jest.mock('../../generic/model-store');
|
jest.mock('../../generic/model-store');
|
||||||
jest.mock('@edx/frontend-platform/auth');
|
jest.mock('@edx/frontend-platform/auth');
|
||||||
getConfig.mockImplementation(() => ({ ENABLE_JUMPNAV: 'true' }));
|
|
||||||
getAuthenticatedUser.mockImplementation(() => ({ administrator: true }));
|
getAuthenticatedUser.mockImplementation(() => ({ administrator: true }));
|
||||||
// ^^^^Remove When Fully rolled out
|
// ^^^^Remove When Fully rolled out
|
||||||
|
|
||||||
|
|||||||
@@ -95,7 +95,6 @@ initialize({
|
|||||||
CONTACT_URL: process.env.CONTACT_URL || null,
|
CONTACT_URL: process.env.CONTACT_URL || null,
|
||||||
CREDENTIALS_BASE_URL: process.env.CREDENTIALS_BASE_URL || null,
|
CREDENTIALS_BASE_URL: process.env.CREDENTIALS_BASE_URL || null,
|
||||||
ENTERPRISE_LEARNER_PORTAL_HOSTNAME: process.env.ENTERPRISE_LEARNER_PORTAL_HOSTNAME || null,
|
ENTERPRISE_LEARNER_PORTAL_HOSTNAME: process.env.ENTERPRISE_LEARNER_PORTAL_HOSTNAME || null,
|
||||||
ENABLE_JUMPNAV: process.env.ENABLE_JUMPNAV || null,
|
|
||||||
ENABLE_NOTICES: process.env.ENABLE_NOTICES || null,
|
ENABLE_NOTICES: process.env.ENABLE_NOTICES || null,
|
||||||
INSIGHTS_BASE_URL: process.env.INSIGHTS_BASE_URL || null,
|
INSIGHTS_BASE_URL: process.env.INSIGHTS_BASE_URL || null,
|
||||||
SEARCH_CATALOG_URL: process.env.SEARCH_CATALOG_URL || null,
|
SEARCH_CATALOG_URL: process.env.SEARCH_CATALOG_URL || null,
|
||||||
|
|||||||
Reference in New Issue
Block a user