Skip to content

feat: restrict trial expired users - #784

Open
suvanbanerjee wants to merge 4 commits into
developmentfrom
feat/restrict-trial-expired-users
Open

suvanbanerjee wants to merge 4 commits into
developmentfrom
feat/restrict-trial-expired-users

Conversation

@suvanbanerjee

@suvanbanerjee suvanbanerjee commented Aug 27, 2026

Copy link
Copy Markdown
Member

Pull Request

Description

restrict trial expired users to past forecast only
image
image
image

Fixes https://github.com/openclimatefix/client-private/issues/500

How Has This Been Tested?

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@suvanbanerjee
suvanbanerjee requested a review from braddf August 27, 2026 02:42
@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nowcasting-app Ready Ready Preview Aug 27, 2026 2:42am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
quartz-app Ignored Ignored Aug 27, 2026 2:42am

Request Review

@suvanbanerjee

Copy link
Copy Markdown
Member Author

should be merged/tested again after openclimatefix/quartz-api#365

@suvanbanerjee suvanbanerjee self-assigned this Aug 27, 2026
@peterdudfield

Copy link
Copy Markdown
Contributor

Thanks for this. A few questions from me

  • What link does the "subscribe to get more" go to?
  • Are these changes just done in the UI, because API users might then still have access - I guess that could be a different PR in the quartz-api
  • What happens if people press the right arrow key? does that go along? (that might be sorted if we do something in the API)

@suvanbanerjee

Copy link
Copy Markdown
Member Author

Thanks @peterdudfield for the questions,

  1. It will open the mail to support@quartz.solar
  2. The API level is also blocked. There’s a PR for that here: feat: add trial check and don't allow forecast access quartz-api#365
  3. It will only go up to the current point in time (NOW). Anything after that isn’t a real forecast, it’s just mock data, so the right arrow won’t take users beyond NOW.

@peterdudfield

Copy link
Copy Markdown
Contributor

Thanks @peterdudfield for the questions,

  1. It will open the mail to support@quartz.solar
  2. The API level is also blocked. There’s a PR for that here: feat: add trial check and don't allow forecast access quartz-api#365
  3. It will only go up to the current point in time (NOW). Anything after that isn’t a real forecast, it’s just mock data, so the right arrow won’t take users beyond NOW.

great stuff! You've got it all covered then!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants