Skip to content

Commit f1dab90

Browse files
committed
Address PR comments
1 parent 4c55129 commit f1dab90

File tree

3 files changed

+9
-10
lines changed

3 files changed

+9
-10
lines changed

public/video-ui/src/components/VideoUpload/AddSelfHostedAsset.jsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ export default class AddSelfHostedAsset extends React.Component {
1717
};
1818

1919
render() {
20-
const { video, permissions, uploading, startUpload } = this.props;
20+
const { video, permissions, isUploading, startUpload } = this.props;
2121

2222
if (!permissions || !permissions.addSelfHostedAsset) {
2323
return false;
@@ -34,13 +34,13 @@ export default class AddSelfHostedAsset extends React.Component {
3434
className="form__field__file"
3535
type="file"
3636
onChange={this.setFile}
37-
disabled={uploading}
37+
disabled={isUploading}
3838
accept="video/*,.mxf"
3939
/>
4040
<button
4141
type="button"
4242
className="btn button__secondary__assets"
43-
disabled={!this.state.file || uploading}
43+
disabled={!this.state.file || isUploading}
4444
onClick={() =>
4545
startUpload({
4646
id: video.id,

public/video-ui/src/components/VideoUpload/YoutubeUpload.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export default class YoutubeUpload extends React.Component {
3333
className="form__field__file"
3434
type="file"
3535
onChange={this.setFile}
36-
disabled={!canUploadToYouTube || this.props.uploading}
36+
disabled={!canUploadToYouTube || this.props.isUploading}
3737
accept="video/*,.mxf"
3838
/>
3939
{ !canUploadToYouTube ?
@@ -44,7 +44,7 @@ export default class YoutubeUpload extends React.Component {
4444
<button
4545
type="button"
4646
className="btn button__secondary__assets"
47-
disabled={!canUploadToYouTube || this.props.uploading || !this.state.file}
47+
disabled={!canUploadToYouTube || this.props.isUploading || !this.state.file}
4848
onClick={() =>
4949
startUpload({
5050
id: video.id,

public/video-ui/src/pages/Upload/index.tsx

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@ import YoutubeUpload from '../../components/VideoUpload/YoutubeUpload';
1414
import { startVideoUpload } from '../../slices/s3Upload';
1515
import { fetchCategories, fetchChannels } from '../../slices/youtube';
1616
import { AppDispatch, RootState } from '../../util/setupStore';
17-
import { getStore } from '../../util/storeAccessor';
1817

1918
export const VideoUpload = (props: { params: { id: string } }) => {
2019
const dispatch = useDispatch<AppDispatch>();
@@ -61,7 +60,7 @@ export const VideoUpload = (props: { params: { id: string } }) => {
6160
store.youtube.channels
6261
]);
6362

64-
const uploading = store.s3Upload.status === 'uploading';
63+
const isUploading = store.s3Upload.status === 'uploading';
6564

6665
const projectId = store.video.plutoData?.projectId;
6766

@@ -80,7 +79,7 @@ export const VideoUpload = (props: { params: { id: string } }) => {
8079
video={store.video}
8180
categories={store.youtube.categories}
8281
channels={store.youtube.channels}
83-
uploading={uploading}
82+
isUploading={isUploading}
8483
saveVideo={bindActionCreators(saveVideo, dispatch)}
8584
startUpload={bindActionCreators(startVideoUpload, dispatch)}
8685
/>
@@ -90,8 +89,8 @@ export const VideoUpload = (props: { params: { id: string } }) => {
9089
/>
9190
<AddSelfHostedAsset
9291
video={store.video}
93-
permissions={getStore().getState().config.permissions}
94-
uploading={uploading}
92+
permissions={store.config.permissions}
93+
isUploading={isUploading}
9594
startUpload={bindActionCreators(startVideoUpload, dispatch)}
9695
/>
9796
</div>

0 commit comments

Comments
 (0)