Skip to content

fix: Add Back button to Pricing page #398

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.

Sign up for GitHub

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

Open
AkashJana18 wants to merge 5 commits into
base: main
Choose a base branch
from
+27 −10
Open

fix: Add Back button to Pricing page #398

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
22016c1
fix: Added Back(logout) button on Pricing Page
Sep 1, 2025
90b254f
fix: Matching the storacha UI
Sep 1, 2025
907ab1e
Merge branch 'storacha:main' into main
AkashJana18 Sep 6, 2025
b33cd14
Merge branch 'storacha:main' into main
AkashJana18 Sep 11, 2025
cf9fd97
chore: Removing unnecessary reformatting by prettier
AkashJana18 Sep 11, 2025
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
9 changes: 8 additions & 1 deletion packages/console/src/components/PlanGate.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import { useConditionalPlan } from '@/hooks'
import { useSearchParams } from 'next/navigation'
import { useIframe } from '@/contexts/IframeContext'
import { useRecordRefcode } from '@/lib/referrals/hooks'
import Link from 'next/link'

const PricingTable = ({
email,
Expand All @@ -20,10 +21,16 @@ const PricingTable = ({
const { isIframe } = useIframe()

return (
<div className="flex flex-col justify-center items-center min-h-screen">
<div className="flex flex-col justify-center items-center min-h-screen relative">
<div className="my-6">
<Logo />
</div>
<Link
href="https://storacha.console.network/logout"
Copy link
Member

@travis travis Sep 24, 2025

Choose a reason for hiding this comment

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

this URL should be different per-environment - in fact, does it even need to be an absolute URL? why not use a relative URL here?

className="absolute top-4 right-4 bg-hot-red text-white px-4 py-2 rounded-full shadow-md hover:border-white transition"
>
&larr; Back
Copy link
Member

@travis travis Sep 24, 2025

Choose a reason for hiding this comment

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

"Back" doesn't feel like it gives the user enough context here - I think it would be better to have this button say "Nevermind, Log Me Out" or something along those lines - I get that the only sensible thing to do if they want to leave this page is to log out, but "Back" makes it sound like we're going to do something else and I want to be super clear with the user about what they're doing.

</Link>
<div className="max-w-screen-lg font-epilogue text-black text-center bg-white border border-hot-red rounded-2xl overflow-hidden p5 mx-4 mb-4">
{referredBy ? (
<>
Expand Down
28 changes: 19 additions & 9 deletions pnpm-lock.yaml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading