Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Account Restricted #1589

Merged
merged 20 commits into from
Sep 29, 2021
Merged
Show file tree
Hide file tree
Changes from 18 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
49 changes: 49 additions & 0 deletions packages/files-ui/src/Components/Elements/RestrictedModeBanner.tsx
Original file line number Diff line number Diff line change
@@ -0,0 +1,49 @@
import { Typography, Button, useHistory } from "@chainsafe/common-components"
import { createStyles, makeStyles, useThemeSwitcher } from "@chainsafe/common-theme"
import { Trans } from "@lingui/macro"
import React from "react"
import { CSFTheme } from "../../Themes/types"
import { ROUTE_LINKS } from "../FilesRoutes"

const useStyles = makeStyles(
({ breakpoints, constants, palette }: CSFTheme) => {
return createStyles({
accountRestrictedNotification: {
position: "fixed",
bottom: 0,
backgroundColor: palette.additional["gray"][10],
color: palette.additional["gray"][1],
padding: "16px 24px",
FSM1 marked this conversation as resolved.
Show resolved Hide resolved
left: 0,
width: "100vw",
[breakpoints.up("md")]: {
left: `${constants.navWidth}px`,
width:`calc(100vw - ${constants.navWidth}px)`,
display: "flex",
justifyContent: "space-between",
alignItems: "center"
}
}
})
}
)

const RestrictedModeBanner = () => {
const classes = useStyles()
const { desktop } = useThemeSwitcher()
const { redirect } = useHistory()

return (
<div className={classes.accountRestrictedNotification}>
<Typography variant={desktop ? "body1" : "body2"}>
<Trans>You&apos;ve got a payment due. Until you&apos;ve settled up, we&apos;ve placed your account in restricted mode</Trans>
</Typography>
<Button
onClick={() => redirect(ROUTE_LINKS.SettingsPath("plan"))}
fullsize={!desktop}>
<Trans>Go to Payments</Trans>
</Button>
</div>)
}

export default RestrictedModeBanner
13 changes: 6 additions & 7 deletions packages/files-ui/src/Components/Layouts/AppWrapper.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ const useStyles = makeStyles(
padding: "0",
"&.active": {
// This moves the content areas based on the size of the nav bar

padding: `${0}px ${constants.contentPadding}px ${0}px ${
Number(constants.navWidth) +
Number(constants.contentPadding)
Expand All @@ -34,20 +33,18 @@ const useStyles = makeStyles(
[breakpoints.down("md")]: {}
},
content: {
height: "initial",
[breakpoints.up("md")]: {
height: "100%",
minHeight: "100vh",
transitionDuration: `${animation.translate}ms`,
padding: 0,
"&.active": {
height: "initial",
padding: `${constants.contentTopPadding}px 0 0`
}
},
[breakpoints.down("md")]: {
minHeight: "100vh",
"&.active": {
height: "initial",
padding: `${constants.mobileHeaderHeight}px 0 0`
}
}
Expand Down Expand Up @@ -84,14 +81,16 @@ const AppWrapper: React.FC<IAppWrapper> = ({ children }: IAppWrapper) => {
setNavOpen={setNavOpen}
/>
<section
className={clsx(classes.content, {
active:
className={clsx(
classes.content, {
active:
isLoggedIn &&
secured &&
!!publicKey &&
!isNewDevice &&
!shouldInitializeAccount
})}
}
)}
>
{children}
</section>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@ import getFilesFromDataTransferItems from "../../../Utils/getFilesFromDataTransf

const CSFFileBrowser: React.FC<IFileBrowserModuleProps> = () => {
const { downloadFile, uploadFiles, buckets } = useFiles()
const { accountRestricted } = useFilesApi()
const { filesApiClient } = useFilesApi()
const { addToast } = useToasts()
const [loadingCurrentPath, setLoadingCurrentPath] = useState(false)
Expand Down Expand Up @@ -92,12 +93,11 @@ const CSFFileBrowser: React.FC<IFileBrowserModuleProps> = () => {
const message = `${
itemToDelete.isFolder ? t`Folder` : t`File`
} ${t`deleted successfully`}`
const id = addToast({
addToast({
title: message,
type: "success",
testId: "deletion-success"
})
console.log(id)
}
return Promise.resolve()
} catch (error) {
Expand Down Expand Up @@ -176,12 +176,20 @@ const CSFFileBrowser: React.FC<IFileBrowserModuleProps> = () => {

const handleUploadOnDrop = useCallback(async (files: File[], fileItems: DataTransferItemList, path: string) => {
if (!bucket) return
if (accountRestricted) {
addToast({
type:"error",
title: t`Uploads disabled`,
subtitle: t`Oops! You need to pay for this month to upload more content.`
})
return
}
const flattenedFiles = await getFilesFromDataTransferItems(fileItems)
const paths = [...new Set(flattenedFiles.map(f => f.filepath))]
paths.forEach(p => {
uploadFiles(bucket, flattenedFiles.filter(f => f.filepath === p), getPathWithFile(path, p))
})
}, [uploadFiles, bucket])
}, [uploadFiles, bucket, accountRestricted, addToast])

const viewFolder = useCallback((cid: string) => {
const fileSystemItem = pathContents.find(f => f.cid === cid)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import { useFileBrowser } from "../../../Contexts/FileBrowserContext"
import clsx from "clsx"
import { useEffect } from "react"
import { nameValidator } from "../../../Utils/validationSchema"
import { useFilesApi } from "../../../Contexts/FilesApiContext"

const useStyles = makeStyles(
({ breakpoints, constants, palette, typography, zIndex }: CSFTheme) => {
Expand Down Expand Up @@ -188,6 +189,7 @@ interface IShareFileProps {
const ShareModal = ({ close, file, filePath }: IShareFileProps) => {
const classes = useStyles()
const { handleCreateSharedFolder } = useCreateOrEditSharedFolder()
const { accountRestricted } = useFilesApi()
const [sharedFolderName, setSharedFolderName] = useState("")
const { sharedFolderReaders, sharedFolderWriters, handleLookupUser, onNewUsers, usersError, resetUsers } = useLookupSharedFolderUser()
const [isUsingCurrentBucket, setIsUsingCurrentBucket] = useState(true)
Expand All @@ -213,14 +215,16 @@ const ShareModal = ({ close, file, filePath }: IShareFileProps) => {
.filter(buck => buck.type === "share" || buck.type === "csf")
// filter out the current bucket
.filter(buck => buck.id !== bucket?.id)
// all buckets where the user is reader or writer
// all buckets where the user is owner or writer
.filter(buck => !!buck.writers.find((w) => w.uuid === profile.userId) || !!buck.owners.find((o) => o.uuid === profile.userId))
// filter out CSF and share buckets where user is an owner if their account is restricted
.filter(buck => !(!!accountRestricted && (buck.type === "csf" || !!buck.owners.find(o => o.uuid === profile.userId))))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer the owner's shared folders to be disabled !
Maybe we can have a separate ticket for that !

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ahhh yeah that is probably doable, though might be a bit confusing for users why certain items are enabled while others are not.

.map(buck => ({
label: buck.name || t`Home`,
value: buck.id
}))
}
, [bucket, buckets, profile])
, [bucket, buckets, profile, accountRestricted])

const hasNoSharedBucket = useMemo(() => bucketsOptions.length === 0, [bucketsOptions.length])

Expand All @@ -232,10 +236,10 @@ const ShareModal = ({ close, file, filePath }: IShareFileProps) => {

// if the user has no shared bucket, we default to new folder creation
useEffect(() => {
if (hasNoSharedBucket) {
if (hasNoSharedBucket && !accountRestricted) {
setIsUsingCurrentBucket(false)
}
}, [hasNoSharedBucket])
}, [hasNoSharedBucket, accountRestricted])

const onNameChange = useCallback((value?: string | number) => {
if (value === undefined) return
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,11 @@ import { useFilesApi } from "../../../Contexts/FilesApiContext"
import { useUser } from "../../../Contexts/UserContext"
import DragAndDrop from "../../../Contexts/DnDContext"
import FilesList from "./views/FilesList"
import getFilesFromDataTransferItems from "../../../Utils/getFilesFromDataTransferItems"

const SharedFileBrowser = () => {
const { downloadFile, uploadFiles, buckets, refreshBuckets, getStorageSummary } = useFiles()
const { filesApiClient } = useFilesApi()
const { filesApiClient, accountRestricted } = useFilesApi()
const { addToast } = useToasts()
const [loadingCurrentPath, setLoadingCurrentPath] = useState(false)
const [pathContents, setPathContents] = useState<FileSystemItem[]>([])
Expand Down Expand Up @@ -179,23 +180,20 @@ const SharedFileBrowser = () => {

const handleUploadOnDrop = useCallback(async (files: File[], fileItems: DataTransferItemList, path: string) => {
if (!bucket) return
let hasFolder = false
for (let i = 0; i < files.length; i++) {
if (fileItems[i].webkitGetAsEntry()?.isDirectory) {
hasFolder = true
}
}
if (hasFolder) {
if (accountRestricted) {
addToast({
title: t`Folder uploads are not supported currently`,
type: "error"
type:"error",
title: t`Unable to upload`,
subtitle: t`Oops! You need to pay for this month to upload more content.`
})
} else {
uploadFiles(bucket, files, path)
.then(() => refreshContents())
.catch(console.error)
return
}
}, [addToast, uploadFiles, bucket, refreshContents])
const flattenedFiles = await getFilesFromDataTransferItems(fileItems)
FSM1 marked this conversation as resolved.
Show resolved Hide resolved
const paths = [...new Set(flattenedFiles.map(f => f.filepath))]
paths.forEach(p => {
uploadFiles(bucket, flattenedFiles.filter(f => f.filepath === p), getPathWithFile(path, p))
})
}, [uploadFiles, bucket, accountRestricted, addToast])

const bulkOperations: IBulkOperations = useMemo(() => ({
[CONTENT_TYPES.Directory]: ["download", "move", "delete"],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ import { SharedFolderModalMode } from "./types"
import SharingExplainerModal from "../../SharingExplainerModal"
import { useSharingExplainerModalFlag } from "./hooks/useSharingExplainerModalFlag"
import { usePageTrack } from "../../../Contexts/PosthogContext"
import RestrictedModeBanner from "../../Elements/RestrictedModeBanner"

export const desktopSharedGridSettings = "69px 3fr 120px 190px 150px 45px !important"
export const mobileSharedGridSettings = "3fr 80px 45px !important"
Expand All @@ -38,7 +39,10 @@ const useStyles = makeStyles(
position: "relative",
[breakpoints.down("md")]: {
marginLeft: constants.generalUnit * 2,
marginRight: constants.generalUnit * 2
marginRight: constants.generalUnit * 2,
"&.bottomBanner": {
paddingBottom: constants.bottomBannerMobileHeight
}
},
[breakpoints.up("md")]: {
border: "1px solid transparent",
Expand All @@ -47,6 +51,10 @@ const useStyles = makeStyles(
minHeight: `calc(100vh - ${Number(constants.contentTopPadding)}px)`,
"&.droppable": {
borderColor: palette.additional["geekblue"][4]
},
"&.bottomBanner": {
minHeight: `calc(100vh - ${Number(constants.contentTopPadding) + Number(constants.bottomBannerHeight)}px)`,
paddingBottom: constants.bottomBannerHeight
}
}
},
Expand Down Expand Up @@ -108,7 +116,7 @@ type SortingType = "name" | "size" | "date_uploaded"

const SharedFolderOverview = () => {
const classes = useStyles()
const { filesApiClient } = useFilesApi()
const { filesApiClient, accountRestricted } = useFilesApi()
const { buckets, isLoadingBuckets, refreshBuckets } = useFiles()
const [createOrEditSharedFolderMode, setCreateOrEditSharedFolderMode] = useState<SharedFolderModalMode | undefined>(undefined)
const [bucketToEdit, setBucketToEdit] = useState<BucketKeyPermission | undefined>(undefined)
Expand Down Expand Up @@ -160,10 +168,13 @@ const SharedFolderOverview = () => {
const openSharedFolder = useCallback((bucketId: string) => {
redirect(ROUTE_LINKS.SharedFolderExplorer(bucketId, "/"))
}, [redirect])

return (
<>
<article
className={classes.root}
className={clsx(classes.root, {
bottomBanner: accountRestricted
})}
>
<header className={classes.header}>
<Typography
Expand All @@ -180,6 +191,7 @@ const SharedFolderOverview = () => {
setBucketToEdit(undefined)
setCreateOrEditSharedFolderMode("create")
}}
disabled={accountRestricted}
>
<PlusIcon />
<Trans>Create a Shared Folder</Trans>
Expand Down Expand Up @@ -294,6 +306,9 @@ const SharedFolderOverview = () => {
injectedClass={{ inner: classes.confirmDeletionDialog }}
testId="shared-folder-deletion"
/>
{accountRestricted &&
<RestrictedModeBanner />
}
</>
)
}
Expand Down
Loading