refactor: rename DisplayPictureModal to EditProfilePictureModal

pull/2765/head
William Grant 2 years ago
parent b2c473c489
commit 977dbea9dc

@ -7,7 +7,7 @@ import { SyncUtils, ToastUtils, UserUtils } from '../../session/utils';
import { YourSessionIDPill, YourSessionIDSelectable } from '../basic/YourSessionIDPill'; import { YourSessionIDPill, YourSessionIDSelectable } from '../basic/YourSessionIDPill';
import { getConversationController } from '../../session/conversations'; import { getConversationController } from '../../session/conversations';
import { editProfileModal, updateDisplayPictureModel } from '../../state/ducks/modalDialog'; import { editProfileModal, updateEditProfilePictureModel } from '../../state/ducks/modalDialog';
import { saveQRCode } from '../../util/saveQRCode'; import { saveQRCode } from '../../util/saveQRCode';
import { setLastProfileUpdateTimestamp } from '../../util/storage'; import { setLastProfileUpdateTimestamp } from '../../util/storage';
import { SessionWrapperModal } from '../SessionWrapperModal'; import { SessionWrapperModal } from '../SessionWrapperModal';
@ -191,7 +191,7 @@ export const EditProfileDialog = (): ReactElement => {
const handleProfileHeaderClick = () => { const handleProfileHeaderClick = () => {
closeDialog(); closeDialog();
dispatch( dispatch(
updateDisplayPictureModel({ updateEditProfilePictureModel({
avatarPath, avatarPath,
profileName, profileName,
ourId, ourId,

@ -3,7 +3,7 @@ import { SessionWrapperModal } from '../SessionWrapperModal';
import { SessionButton, SessionButtonColor, SessionButtonType } from '../basic/SessionButton'; import { SessionButton, SessionButtonColor, SessionButtonType } from '../basic/SessionButton';
import { SpacerLG } from '../basic/Text'; import { SpacerLG } from '../basic/Text';
import { useDispatch } from 'react-redux'; import { useDispatch } from 'react-redux';
import { editProfileModal, updateDisplayPictureModel } from '../../state/ducks/modalDialog'; import { editProfileModal, updateEditProfilePictureModel } from '../../state/ducks/modalDialog';
import { ProfileAvatar } from './EditProfileDialog'; import { ProfileAvatar } from './EditProfileDialog';
import styled from 'styled-components'; import styled from 'styled-components';
import { clearOurAvatar, uploadOurAvatar } from '../../interactions/conversationInteractions'; import { clearOurAvatar, uploadOurAvatar } from '../../interactions/conversationInteractions';
@ -60,13 +60,13 @@ const uploadProfileAvatar = async (scaledAvatarUrl: string | null) => {
} }
}; };
export type DisplayPictureModalProps = { export type EditProfilePictureModalProps = {
avatarPath: string | null; avatarPath: string | null;
profileName: string | undefined; profileName: string | undefined;
ourId: string; ourId: string;
}; };
export const DisplayPictureModal = (props: DisplayPictureModalProps) => { export const EditProfilePictureModal = (props: EditProfilePictureModalProps) => {
const dispatch = useDispatch(); const dispatch = useDispatch();
if (!props) { if (!props) {
@ -79,7 +79,7 @@ export const DisplayPictureModal = (props: DisplayPictureModalProps) => {
const [loading, setLoading] = useState(false); const [loading, setLoading] = useState(false);
const closeDialog = () => { const closeDialog = () => {
dispatch(updateDisplayPictureModel(null)); dispatch(updateEditProfilePictureModel(null));
dispatch(editProfileModal({})); dispatch(editProfileModal({}));
}; };
@ -99,7 +99,7 @@ export const DisplayPictureModal = (props: DisplayPictureModalProps) => {
await uploadProfileAvatar(newAvatarObjectUrl); await uploadProfileAvatar(newAvatarObjectUrl);
setLoading(false); setLoading(false);
dispatch(updateDisplayPictureModel(null)); dispatch(updateEditProfilePictureModel(null));
}; };
const handleRemove = async () => { const handleRemove = async () => {
@ -107,7 +107,7 @@ export const DisplayPictureModal = (props: DisplayPictureModalProps) => {
await clearOurAvatar(); await clearOurAvatar();
setNewAvatarObjectUrl(null); setNewAvatarObjectUrl(null);
setLoading(false); setLoading(false);
dispatch(updateDisplayPictureModel(null)); dispatch(updateEditProfilePictureModel(null));
}; };
return ( return (

@ -7,7 +7,7 @@ import {
getChangeNickNameDialog, getChangeNickNameDialog,
getConfirmModal, getConfirmModal,
getDeleteAccountModalState, getDeleteAccountModalState,
getDisplayPictureModalState, getEditProfilePictureModalState,
getEditProfileDialog, getEditProfileDialog,
getInviteContactModal, getInviteContactModal,
getOnionPathDialog, getOnionPathDialog,
@ -37,7 +37,7 @@ import { SessionNicknameDialog } from './SessionNicknameDialog';
import { BanOrUnBanUserDialog } from './BanOrUnbanUserDialog'; import { BanOrUnBanUserDialog } from './BanOrUnbanUserDialog';
import { ReactListModal } from './ReactListModal'; import { ReactListModal } from './ReactListModal';
import { ReactClearAllModal } from './ReactClearAllModal'; import { ReactClearAllModal } from './ReactClearAllModal';
import { DisplayPictureModal } from './DisplayPictureModal'; import { EditProfilePictureModal } from './EditProfilePictureModal';
export const ModalContainer = () => { export const ModalContainer = () => {
const confirmModalState = useSelector(getConfirmModal); const confirmModalState = useSelector(getConfirmModal);
@ -57,7 +57,7 @@ export const ModalContainer = () => {
const banOrUnbanUserModalState = useSelector(getBanOrUnbanUserModalState); const banOrUnbanUserModalState = useSelector(getBanOrUnbanUserModalState);
const reactListModalState = useSelector(getReactListDialog); const reactListModalState = useSelector(getReactListDialog);
const reactClearAllModalState = useSelector(getReactClearAllDialog); const reactClearAllModalState = useSelector(getReactClearAllDialog);
const DisplayPictureModalState = useSelector(getDisplayPictureModalState); const EditProfilePictureModalState = useSelector(getEditProfilePictureModalState);
return ( return (
<> <>
@ -82,7 +82,9 @@ export const ModalContainer = () => {
{confirmModalState && <SessionConfirm {...confirmModalState} />} {confirmModalState && <SessionConfirm {...confirmModalState} />}
{reactListModalState && <ReactListModal {...reactListModalState} />} {reactListModalState && <ReactListModal {...reactListModalState} />}
{reactClearAllModalState && <ReactClearAllModal {...reactClearAllModalState} />} {reactClearAllModalState && <ReactClearAllModal {...reactClearAllModalState} />}
{DisplayPictureModalState && <DisplayPictureModal {...DisplayPictureModalState} />} {EditProfilePictureModalState && (
<EditProfilePictureModal {...EditProfilePictureModalState} />
)}
</> </>
); );
}; };

@ -1,7 +1,7 @@
import { createSlice, PayloadAction } from '@reduxjs/toolkit'; import { createSlice, PayloadAction } from '@reduxjs/toolkit';
import { SessionConfirmDialogProps } from '../../components/dialog/SessionConfirm'; import { SessionConfirmDialogProps } from '../../components/dialog/SessionConfirm';
import { PasswordAction } from '../../components/dialog/SessionPasswordDialog'; import { PasswordAction } from '../../components/dialog/SessionPasswordDialog';
import { DisplayPictureModalProps } from '../../components/dialog/DisplayPictureModal'; import { EditProfilePictureModalProps } from '../../components/dialog/EditProfilePictureModal';
export type BanType = 'ban' | 'unban'; export type BanType = 'ban' | 'unban';
export type ConfirmModalState = SessionConfirmDialogProps | null; export type ConfirmModalState = SessionConfirmDialogProps | null;
@ -35,7 +35,7 @@ export type ReactModalsState = {
messageId: string; messageId: string;
} | null; } | null;
export type DisplayPictureModalState = DisplayPictureModalProps | null; export type EditProfilePictureModalState = EditProfilePictureModalProps | null;
export type ModalState = { export type ModalState = {
confirmModal: ConfirmModalState; confirmModal: ConfirmModalState;
@ -55,7 +55,7 @@ export type ModalState = {
deleteAccountModal: DeleteAccountModalState; deleteAccountModal: DeleteAccountModalState;
reactListModalState: ReactModalsState; reactListModalState: ReactModalsState;
reactClearAllModalState: ReactModalsState; reactClearAllModalState: ReactModalsState;
displayPictureModalState: DisplayPictureModalState; displayPictureModalState: EditProfilePictureModalState;
}; };
export const initialModalState: ModalState = { export const initialModalState: ModalState = {
@ -134,7 +134,7 @@ const ModalSlice = createSlice({
updateReactClearAllModal(state, action: PayloadAction<ReactModalsState>) { updateReactClearAllModal(state, action: PayloadAction<ReactModalsState>) {
return { ...state, reactClearAllModalState: action.payload }; return { ...state, reactClearAllModalState: action.payload };
}, },
updateDisplayPictureModel(state, action: PayloadAction<DisplayPictureModalState>) { updateEditProfilePictureModel(state, action: PayloadAction<EditProfilePictureModalState>) {
return { ...state, displayPictureModalState: action.payload }; return { ...state, displayPictureModalState: action.payload };
}, },
}, },
@ -159,6 +159,6 @@ export const {
updateBanOrUnbanUserModal, updateBanOrUnbanUserModal,
updateReactListModal, updateReactListModal,
updateReactClearAllModal, updateReactClearAllModal,
updateDisplayPictureModel, updateEditProfilePictureModel,
} = actions; } = actions;
export const modalReducer = reducer; export const modalReducer = reducer;

@ -8,7 +8,7 @@ import {
ChangeNickNameModalState, ChangeNickNameModalState,
ConfirmModalState, ConfirmModalState,
DeleteAccountModalState, DeleteAccountModalState,
DisplayPictureModalState, EditProfilePictureModalState,
EditProfileModalState, EditProfileModalState,
InviteContactModalState, InviteContactModalState,
ModalState, ModalState,
@ -111,7 +111,7 @@ export const getReactClearAllDialog = createSelector(
(state: ModalState): ReactModalsState => state.reactClearAllModalState (state: ModalState): ReactModalsState => state.reactClearAllModalState
); );
export const getDisplayPictureModalState = createSelector( export const getEditProfilePictureModalState = createSelector(
getModal, getModal,
(state: ModalState): DisplayPictureModalState => state.displayPictureModalState (state: ModalState): EditProfilePictureModalState => state.displayPictureModalState
); );

Loading…
Cancel
Save