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

Remove internal tag routes #947

Closed
wants to merge 2 commits into from
Closed

Remove internal tag routes #947

wants to merge 2 commits into from

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented Aug 18, 2016

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

@rullzer rullzer added the 2. developing Work in progress label Aug 18, 2016
@rullzer rullzer added this to the Nextcloud 11.0 milestone Aug 18, 2016
@mention-bot
Copy link

@rullzer, thanks for your PR! By analyzing the annotation information on this pull request, we identified @tanghus, @nickvergessen and @MorrisJobke to be potential reviewers

@rullzer
Copy link
Member Author

rullzer commented Aug 18, 2016

If it is used we should move it to a real (tested) controller. But I could find no such things.

@nickvergessen
Copy link
Member

Those are the favorite icon tags?

@rullzer
Copy link
Member Author

rullzer commented Aug 19, 2016

No that is done via a route in the files app

@nickvergessen
Copy link
Member

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.

@rullzer
Copy link
Member Author

rullzer commented Aug 19, 2016

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.

@rullzer rullzer closed this Aug 19, 2016
@rullzer rullzer deleted the remove_tags branch August 19, 2016 06:19
@nickvergessen
Copy link
Member

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 😉

@rullzer
Copy link
Member Author

rullzer commented Aug 19, 2016

Well then other apps should not use it as well ;)

@MorrisJobke
Copy link
Member

Well then other apps should not use it as well ;)

Correct.

@rullzer
Copy link
Member Author

rullzer commented Aug 19, 2016

I moved it over to a real Controller in #958
But lets deprecate the routes and remove them in NC12 or so?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants