Skip to content

feat: Shared with who#75

Merged
AndreaVelasquezA merged 9 commits intodevfrom
feat/shared_with_who
Oct 20, 2023
Merged

feat: Shared with who#75
AndreaVelasquezA merged 9 commits intodevfrom
feat/shared_with_who

Conversation

@AndreaVelasquezA
Copy link
Copy Markdown
Member

tests implementation reaches more than 70%

Endpoint to get the users with whom a file is shared.
Testing has not been implemented yet
…in addition to implementing the tests

The tests exceed the required 70%
@PedroChaparro PedroChaparro self-requested a review October 20, 2023 11:28
@PedroChaparro PedroChaparro added the feature A new feature label Oct 20, 2023
@PedroChaparro PedroChaparro changed the title Feat/shared with who feat: Shared with who Oct 20, 2023
PedroChaparro and others added 5 commits October 20, 2023 06:52
Return an array of strings instead of an array of:

```json
{
    "username": "string"
}
```
A new test case is added to test the SOAP domain error thrown when a file is not found
Copy link
Copy Markdown
Contributor

@PedroChaparro PedroChaparro left a comment

Choose a reason for hiding this comment

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

Thanks for implementing. Squash and merge 🚀

@AndreaVelasquezA AndreaVelasquezA merged commit 3281cca into dev Oct 20, 2023
@PedroChaparro PedroChaparro deleted the feat/shared_with_who branch October 20, 2023 16:03
@PedroChaparro PedroChaparro mentioned this pull request Oct 20, 2023
PedroChaparro added a commit that referenced this pull request Oct 20, 2023
* docs: Add bruno collection (#77)

* fix: move a file (#78)

* feat: Shared with who (#75)

* feat: Get the users with whom a file is being shared

Endpoint to get the users with whom a file is shared.

* refactor: Endpoint correction in addition to updating the spec

* feat: The init and view are added for the endpoint shared with whom, in addition to implementing the tests

The tests exceed the required 70%

* docs(openapi): Update spec

---------

Co-authored-by: Andvelavi <54145562+Andvelavi@users.noreply.github.com>
Co-authored-by: Pedro Andrés Chaparro Quintero <pedro.chaparro.2020@upb.edu.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants