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 description to public links + internal public links #3305

Merged
merged 16 commits into from
Oct 7, 2022

Conversation

gmgigi96
Copy link
Member

@gmgigi96 gmgigi96 commented Oct 3, 2022

This PR adds the possibility to set a description to a public link, allowing an user to describe the usefulness of the public link.
It also adds an option to set the public links for internal usage (for example for being using by apps). When a public link is internal, this will not be exposed in the list of public links.

@update-docs
Copy link

update-docs bot commented Oct 3, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@gmgigi96 gmgigi96 changed the title Add description to public link Add description to public links Oct 3, 2022
labkode
labkode previously requested changes Oct 3, 2022
Copy link
Member

@labkode labkode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more field in DB hidden(boolean) so the normal shares queries from the web will always be on type: select * from oc_share where type = public link and hidden=false, today we have codimd, but tomorrow we may have other apps and then we would need to touch again the normal query (not good).

@glpatcern
Copy link
Member

As just discussed with Gianmaria: let's call the extra field internal (bool), defaults = false, also in the CS3APIs. Of course will have it "exposed" for users to use it (if they discover/reverse engineer outside the web UI), that's fine after all.

@gmgigi96
Copy link
Member Author

gmgigi96 commented Oct 3, 2022

The hidden in the db requires to modify the cs3apis, as now is not anymore cernbox specific. As we discussed with @glpatcern we could add a bool field named Internal, to specify that that public link is used for internal purpose (in the context of an application, for example), and so because was not created by the user, is hidden in the list. What do you think?

@labkode
Copy link
Member

labkode commented Oct 3, 2022

Go ahead!

@gmgigi96 gmgigi96 force-pushed the publiclinks-tag branch 2 times, most recently from fc8f6b4 to 192974e Compare October 4, 2022 09:27
@gmgigi96 gmgigi96 requested a review from labkode October 5, 2022 07:11
@gmgigi96 gmgigi96 changed the title Add description to public links Add description to public links + internal public links Oct 7, 2022
@glpatcern glpatcern dismissed labkode’s stale review October 7, 2022 15:15

Changes were implemented

@glpatcern glpatcern merged commit 1bfb714 into cs3org:master Oct 7, 2022
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 18, 2022
* use update go cs3apis (temporary)

* Add description in db public link

* add description in the sql driver

* update go cs3apis

* add description in xml response for public links

* create and update a public link with description

* hide tags by description in sql driver

* update cs3apis

* add description during creation

* updated cli cmd for public share description

* fix linting

* fix query to hide tags

* add changelog

* add option to hide public shares

* updated go-cs3apis
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 18, 2022
* use update go cs3apis (temporary)

* Add description in db public link

* add description in the sql driver

* update go cs3apis

* add description in xml response for public links

* create and update a public link with description

* hide tags by description in sql driver

* update cs3apis

* add description during creation

* updated cli cmd for public share description

* fix linting

* fix query to hide tags

* add changelog

* add option to hide public shares

* updated go-cs3apis
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 20, 2022
* use update go cs3apis (temporary)

* Add description in db public link

* add description in the sql driver

* update go cs3apis

* add description in xml response for public links

* create and update a public link with description

* hide tags by description in sql driver

* update cs3apis

* add description during creation

* updated cli cmd for public share description

* fix linting

* fix query to hide tags

* add changelog

* add option to hide public shares

* updated go-cs3apis
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 26, 2022
* use update go cs3apis (temporary)

* Add description in db public link

* add description in the sql driver

* update go cs3apis

* add description in xml response for public links

* create and update a public link with description

* hide tags by description in sql driver

* update cs3apis

* add description during creation

* updated cli cmd for public share description

* fix linting

* fix query to hide tags

* add changelog

* add option to hide public shares

* updated go-cs3apis
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