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

Add StringUtils #1440

Merged
merged 2 commits into from
Dec 31, 2024
Merged

Add StringUtils #1440

merged 2 commits into from
Dec 31, 2024

Conversation

Luna712
Copy link
Contributor

@Luna712 Luna712 commented Dec 29, 2024

This may be completely useless (but did plan to use it myself) so feel free to close if you feel it is absolutely unnecessary. This is meant for use by extensions for URI manipulation since the same method seems to be used across many providers and extractors (search), so these methods were just meant to streamline that.

I also plan to potential expand it with more String methods later which is why it is StringX not URIX.

@fire-light42
Copy link
Collaborator

Why not just place it in AppUtils?

@Luna712
Copy link
Contributor Author

Luna712 commented Dec 29, 2024

Why not just place it in AppUtils?

I thought about it but figured it was cleaner to do a new one here, more clear that it is available and more String related methods could be added here later on. If you want me to change it I can.

@IndusAryan
Copy link
Contributor

@Luna712
Copy link
Contributor Author

Luna712 commented Dec 30, 2024

@fire-light42 fire-light42 merged commit b3a4af1 into recloudstream:master Dec 31, 2024
2 checks passed
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.

3 participants