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

Allow unlimited usage #95

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

buzzclue
Copy link

One can set charges to -1 on consumable feature to make it unlimited.

@bestiony
Copy link

looks interesting . I need this too . maybe you should've had no changes to the style because it's hard to track where your changes actually are

One can set charges to -1 on consumable feature to make it unlimited.

@buzzclue
Copy link
Author

restored code styling :)

@odaiderawi
Copy link

Hello guys, do you plan to merge this PR or what? its looks very good and great idea

@lucasdotvin
Copy link
Owner

Hi! Just to check: do you guys think this behaviour is met by the postpaid features?

https://github.com/lucasdotvin/laravel-soulbscription?tab=readme-ov-file#postpaid-features

@buzzclue
Copy link
Author

buzzclue commented Jul 7, 2024

Hi! Just to check: do you guys think this behaviour is met by the postpaid features?

https://github.com/lucasdotvin/laravel-soulbscription?tab=readme-ov-file#postpaid-features

First, the administrator sets this unlimited value, either manually or through a custom UI.

Now, let's analyze the postpaid plan. If a plan allows n number of usages, the normal consumption process applies. However, if one or more features have the -1 state (indicating unlimited usage), the usage will be tracked but will never be exhausted. Any number of usages will always return true, which is exactly how it is supposed to work.

Below is my implementation of unlimited.

image

@on-future
Copy link

It's a much needed feature!
It is needed independently of postpaid!!!

@imYashGupta
Copy link

guys if it works well , we really needed this feature in this package please Merge ASAP.

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.

6 participants