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

Getting plan interval #222

Closed
PabloHendriks opened this issue Feb 5, 2024 · 3 comments · Fixed by #223
Closed

Getting plan interval #222

PabloHendriks opened this issue Feb 5, 2024 · 3 comments · Fixed by #223
Labels
major release The issue describes something that will be implemented in a future major release

Comments

@PabloHendriks
Copy link
Contributor

Hi,

My problem is about getting the plan interval. When looking at the DefaultIntervalGenerator, the interval is a protected attribute and there is only a isMonthly() method which would not make distinctions between a monthly subscription or a 3-month subscription. Same thing with a yearly or 2 yearly subscription.

Thanks in advance.

@Naoray
Copy link
Collaborator

Naoray commented Feb 7, 2024

Hi @PabloHendriks,

thank you for your contribution to the codebase.

Before I merge your PR I want to make sure that there isn't another way for your specific use case.

What exactly is your use case? What does your cashier_plans.php config file look like? Couldn't you get the information from there?

@PabloHendriks
Copy link
Contributor Author

Hi @Naoray,

Thank you very much for your quick reply.

Yes, I could use the config to access the interval, though it would be nice to way of accessing the interval from the object itself. I believe that a more 'dynamic' way of accessing it would be more interesting.

Let me know what you think!

@sandervanhooft
Copy link
Collaborator

Hi @PabloHendriks,

I am not entirely convinced yet, but we're keeping this open for review for the next major release.

@sandervanhooft sandervanhooft added the major release The issue describes something that will be implemented in a future major release label Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major release The issue describes something that will be implemented in a future major release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants