Merge pull request #30957 from code-dot-org/staging

DTT (Staging > Test) [robo-dtt]

apps/src/code-studio/components/SoundList.jsx

@@ -43,7 +43,7 @@ export default class SoundList extends React.Component {
         this.props.selectedSound.name === sound.name ? true : false;
       return (
         <SoundListEntry
-          key={sound.name}
+          key={sound.sourceUrl}
           assetChosen={this.props.assetChosen}
           soundMetadata={sound}
           isSelected={isSelected}

apps/src/storage/dataBrowser/DataTableView.jsx

@@ -9,7 +9,7 @@ import FontAwesome from '../../templates/FontAwesome';
 import PropTypes from 'prop-types';
 import Radium from 'radium';
 import React from 'react';
-import {changeView, showWarning} from '../redux/data';
+import {changeView, showWarning, tableType} from '../redux/data';
 import * as dataStyles from './dataStyles';
 import color from '../../util/color';
 import {connect} from 'react-redux';
@@ -68,6 +68,7 @@ class DataTableView extends React.Component {
     // from redux state
     tableColumns: PropTypes.arrayOf(PropTypes.string).isRequired,
     tableName: PropTypes.string.isRequired,
+    tableListMap: PropTypes.object.isRequired,
     // "if all of the keys are integers, and more than half of the keys between 0 and
     // the maximum key in the object have non-empty values, then Firebase will render
     // it as an array."
@@ -207,7 +208,12 @@ class DataTableView extends React.Component {
         />
         <div style={debugDataStyle}>{this.getTableJson()}</div>
         {!this.state.showDebugView && (
-          <DataTable getColumnNames={this.getColumnNames} />
+          <DataTable
+            getColumnNames={this.getColumnNames}
+            readOnly={
+              this.props.tableListMap[this.props.tableName] === tableType.SHARED
+            }
+          />
         )}
       </div>
     );
@@ -219,7 +225,8 @@ export default connect(
     view: state.data.view,
     tableColumns: state.data.tableColumns || [],
     tableRecords: state.data.tableRecords || {},
-    tableName: state.data.tableName || ''
+    tableName: state.data.tableName || '',
+    tableListMap: state.data.tableListMap || {}
   }),
   dispatch => ({
     onShowWarning(warningMsg, warningTitle) {