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

Fix/made the dataroom route protected in case user hasn't opted for any trial or plan #613

Merged
merged 2 commits into from
Oct 4, 2024

Conversation

nidhish-srivastava
Copy link
Contributor

@nidhish-srivastava nidhish-srivastava commented Sep 26, 2024

Fixes issue #587
Ensuring that if user hasnt opted for any plan that is he is having free plan and no trial as well then no access to this dataroom route.
If user has lets say business plan but no trial at the moment,he can visit this dataroom page,he shudnt be redirected

Copy link

vercel bot commented Sep 26, 2024

@nidhish-srivastava is attempting to deploy a commit to the mftsio Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

github-actions bot commented Sep 26, 2024

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@nidhish-srivastava nidhish-srivastava changed the title Fix/errorpagehandle Fix/made the dataroom route protected in case user hasn't opted for any trial or plan Sep 26, 2024
@mfts
Copy link
Owner

mfts commented Oct 3, 2024

Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@nidhish-srivastava looks great! finally step before I can merge it in: signing the CLA

@nidhish-srivastava
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@mfts
Copy link
Owner

mfts commented Oct 3, 2024

@nidhish-srivastava are you taking part in oss.gg ? I'd love to assign you 150 Points for it :)

@nidhish-srivastava
Copy link
Contributor Author

Yes i am taking part. It would be so nice of you giving me the points

Copy link

vercel bot commented Oct 3, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
papermark ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 3, 2024 10:57am

@mfts mfts merged commit c385d1c into mfts:main Oct 4, 2024
3 checks passed
Copy link

oss-gg bot commented Oct 4, 2024

Awarding nidhish-srivastava: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/nidhish-srivastava

@github-actions github-actions bot locked and limited conversation to collaborators Oct 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants