search

sailfishos / sailjail-permissions

Other
5 stars 17 forks source link

[permissions] Add Sharing permission details to the README. Fixes JB#58483 #129

Closed llewelld closed 1 year ago

llewelld commented 2 years ago

Adds some info about the Sharing permission for accessing the Share API, specifically to clarify that it is available to all applications by default.

Thaodan commented 2 years ago

Can you mention the version when to are available? Like for example: "Available since version x.x"?

llewelld commented 2 years ago

Can you mention the version when to are available? Like for example: "Available since version x.x"?

The Share API and Sharing permission were both released in 4.2.0 (after the sandboxing was made available in 4.0.1). Given this, I'm thinking this kind of info might be more appropriate in the Share API documentation itself rather than here. What do you think?

Thaodan commented 2 years ago

Can you mention the version when to are available? Like for example: "Available since version x.x"?

The Share API and Sharing permission were both released in 4.2.0 (after the sandboxing was made available in 4.0.1). Given this, I'm thinking this kind of info might be more appropriate in the Share API documentation itself rather than here. What do you think?

Hm it might make sense to have it in both since it is quite useful to have it in this list too, I was thinking of another column that says so for each permission.

Olf0 commented 2 years ago

Hm it might make sense to have it in both since it is quite useful to have it in this list too, I was thinking of another column that says so for each permission.

Two separate places to maintain the same information is usually not a good idea, because they have to be concurrently updated. Usually links are a much better solution, because they alleviate the issue of "updated info here, but missed to update its copy somewhere else".

rainemak commented 1 year ago

@llewelld could you rebase and merge?

llewelld commented 1 year ago

Rebased. Sorry for the delay in getting this merged; I lost track of it.