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.

Sign up for GitHub

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

Jump to bottom

[New architecture] Delete public shares #2517

Closed
21 tasks done
davigonz opened this issue Apr 9, 2019 · 0 comments
Closed
21 tasks done

[New architecture] Delete public shares #2517

davigonz opened this issue Apr 9, 2019 · 0 comments
Labels
Estimation - 3 (M) New architecture
Milestone
2.12.0

Comments

@davigonz
Copy link
Contributor

davigonz commented Apr 9, 2019 •
edited
Loading

New use case, Delete public share for a file

TASKS

  • Include delete public share query in OCShareDao
  • Include delete public share operation in OCLocalDataSource
  • Include delete public share operation OCRemoteDataSource
  • Include delete public share operation in OCShareRepository
  • Include delete public share function in OCShareViewmodel.
  • Connect ShareFileActivity with OCShareViewmodel to delete shares.
  • Error handling, adapt ErrorMessageAdapter
  • Clean up some dependencies:
    • Get shares
    • Create shares
    • Edit shares
    • Remove shares
  • Fix UI tests after cleaning up dependencies in code
  • Use room implementation in Content Provider to delete shares
  • Tests:

TASKS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Estimation - 3 (M) New architecture
Projects
None yet
Milestone
2.12.0
Development

No branches or pull requests
3 participants
@michaelstingl @jesmrec @davigonz