-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Remove internal tag routes #947
Conversation
@rullzer, thanks for your PR! By analyzing the annotation information on this pull request, we identified @tanghus, @nickvergessen and @MorrisJobke to be potential reviewers |
If it is used we should move it to a real (tested) controller. But I could find no such things. |
Those are the favorite icon tags? |
No that is done via a route in the files app |
No, those are the routes, to manually control those tags (like file favorites). It maybe that we don't use them, but any app could use those personal tags. So not sure if we should delete it. I'd vote for moving to a normal controller. |
Fair enough. Should not be to hard. On a side note we need to think how we define 'internal' routes. So stuff that might change at any given time. |
Anything that is not OCS is internal as far as i know 😉 |
Well then other apps should not use it as well ;) |
Correct. |
I moved it over to a real Controller in #958 |
As far as I can tell this is not used at all by any of our internal stuff. It is basically the predecesor of the systemtags as I understand it but is never used.
Any objections? @MorrisJobke @nickvergessen @icewind1991 @schiessle @blizzz @LukasReschke