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.

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

Jump to bottom

Result "status" #122

Closed
TheOfficialDroid opened this issue May 19, 2022 · 3 comments
Closed

Result "status" #122

TheOfficialDroid opened this issue May 19, 2022 · 3 comments

Comments

@TheOfficialDroid
Copy link

I'm going to keep this short, privatising "status" does nothing but add more work on someone using your package, for example getting a users subscribers, $var->status == 200/404, why make it more of a pain? Honestly......
@romanzipp
Copy link
Owner

I'm really sorry for providing a free Open Source package which you can freely use and even modify (wow!).

You could have invested a little bit more time in scrolling through the Result class and would have discovered that there is a getStatus method which will serve your needs. Instead you have used your resources on drafting this disrespectful issue.

@TheOfficialDroid
Copy link
Author

#59

The irony

@romanzipp
Copy link
Owner

So what's you point?

@romanzipp romanzipp closed this as not planned Won't fix, can't repro, duplicate, stale Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@romanzipp @TheOfficialDroid