Revert "fix: show studio button if user has access (#1452)" (#1486)

This reverts commit 82b27e5.
openedx · Sep 13, 2024 · 500e4ab · 500e4ab
1 parent d78851b
commit 500e4ab
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 21 deletions.
diff --git a/src/instructor-toolbar/InstructorToolbar.jsx b/src/instructor-toolbar/InstructorToolbar.jsx
@@ -23,10 +23,10 @@ function getInsightsUrl(courseId) {
   return urlFull;
 }
 
-function getStudioUrl(courseId, unitId, hasStudioAccess) {
+function getStudioUrl(courseId, unitId) {
   const urlBase = getConfig().STUDIO_BASE_URL;
   let urlFull;
-  if (urlBase && hasStudioAccess) {
+  if (urlBase) {
     if (unitId) {
       urlFull = `${urlBase}/container/${unitId}`;
     } else if (courseId) {
@@ -56,11 +56,10 @@ const InstructorToolbar = (props) => {
     courseId,
     unitId,
     tab,
-    hasStudioAccess,
   } = props;
 
   const urlInsights = getInsightsUrl(courseId);
-  const urlStudio = getStudioUrl(courseId, unitId, hasStudioAccess);
+  const urlStudio = getStudioUrl(courseId, unitId);
   const [masqueradeErrorMessage, showMasqueradeError] = useState(null);
 
   const accessExpirationMasqueradeBanner = useAccessExpirationMasqueradeBanner(courseId, tab);
@@ -116,14 +115,12 @@ InstructorToolbar.propTypes = {
   courseId: PropTypes.string,
   unitId: PropTypes.string,
   tab: PropTypes.string,
-  hasStudioAccess: PropTypes.bool,
 };
 
 InstructorToolbar.defaultProps = {
   courseId: undefined,
   unitId: undefined,
   tab: '',
-  hasStudioAccess: false,
 };
 
 export default InstructorToolbar;
diff --git a/src/instructor-toolbar/InstructorToolbar.test.jsx b/src/instructor-toolbar/InstructorToolbar.test.jsx
@@ -34,7 +34,6 @@ describe('Instructor Toolbar', () => {
     mockData = {
       courseId: courseware.courseId,
       unitId: Object.values(models.units)[0].id,
-      hasStudioAccess: true,
     };
     axiosMock.reset();
     axiosMock.onGet(masqueradeUrl).reply(200, { success: true });
@@ -77,16 +76,4 @@ describe('Instructor Toolbar', () => {
 
     expect(screen.queryByText('View course in:')).not.toBeInTheDocument();
   });
-
-  it('does not display Studio link if user does not have studio access', () => {
-    const config = { ...originalConfig };
-    const data = { ...mockData, hasStudioAccess: false };
-    config.INSIGHTS_BASE_URL = 'http://localhost:18100';
-    getConfig.mockImplementation(() => config);
-    render(<InstructorToolbar {...data} />);
-
-    const linksContainer = screen.getByText('View course in:').parentElement;
-    expect(screen.queryByText(linksContainer, 'Studio')).toBeNull();
-    expect(getByText(linksContainer, 'Insights').getAttribute('href')).toMatch(/http.*/);
-  });
 });
diff --git a/src/tab-page/LoadedTabPage.jsx b/src/tab-page/LoadedTabPage.jsx
@@ -26,7 +26,6 @@ const LoadedTabPage = ({
     celebrations,
     org,
     originalUserIsStaff,
-    studioAccess,
     tabs,
     title,
     verifiedMode,
@@ -59,7 +58,6 @@ const LoadedTabPage = ({
           courseId={courseId}
           unitId={unitId}
           tab={activeTabSlug}
-          hasStudioAccess={studioAccess}
         />
       )}
       <StreakModal