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

Using the app to send SMS for other purposes #491

Open
StCyr opened this issue Apr 11, 2022 · 3 comments
Open

Using the app to send SMS for other purposes #491

StCyr opened this issue Apr 11, 2022 · 3 comments

Comments

@StCyr
Copy link

StCyr commented Apr 11, 2022

Hello, I'm looking for an app to send SMS for other purposes than logging in.

It looks like this app would do the trick if I could use the OCA\Service\Gateay\SMS\Gateway class from other apps.

I know using classes from the OCA namespace is generaly not advised but, hey, what are my other choices? ;-)

@vitormattos
Copy link
Contributor

You can use this class but if this class is modified, maybe can break your code.

A solution is contributing with development of this app making the code base best and more stable.

@StCyr
Copy link
Author

StCyr commented Apr 11, 2022

Yes, I can help maintaining this app.

The question is how can I do anything better than using the class from the OCA namespace? I think the only solution is to create an interface in the OCP namespace (hence in nextcloud/server). Am I right?

@vitormattos
Copy link
Contributor

I think that creating an interface in the OCP namespace will be hardy. Imagine the scenario when all app need to implement interface on server repository?

The problem is that you will need follow and help to development of this app to maintain him compatible with your app.

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

No branches or pull requests

2 participants