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 Skeleton for a PublicShareController #6760

Closed
wants to merge 2 commits into from
Closed

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented Oct 4, 2017

This will allow apps to use a generic controller for public pages.
This should allow easier bug fixing and hardening instead of doing it
for each and every app.

@LukasReschke as discussed

TODO:

  • Add actual middleware
  • Move over default share page
  • Tests

@rullzer rullzer added this to the Nextcloud 13 milestone Oct 4, 2017
@rullzer rullzer force-pushed the publicshare-general branch 2 times, most recently from 6a0ac15 to 4e10d43 Compare October 5, 2017 07:53
This will allow apps to use a generic controller for public pages.
This should allow easier bug fixing and hardening instead of doing it
for each and every app.

Signed-off-by: Roeland Jago Douma <[email protected]>
Signed-off-by: Roeland Jago Douma <[email protected]>
@rullzer rullzer force-pushed the publicshare-general branch from 4e10d43 to a99016d Compare October 5, 2017 07:56
@rullzer rullzer modified the milestones: Nextcloud 13, Nextcloud 14 Nov 2, 2017
@rullzer
Copy link
Member Author

rullzer commented Nov 2, 2017

I need to think and polish this more. I'll won't do that this week so lets move it to 14

@juliusknorr juliusknorr mentioned this pull request Jan 29, 2018
2 tasks
@MorrisJobke
Copy link
Member

@rullzer still the way to go?

@rullzer
Copy link
Member Author

rullzer commented Mar 7, 2018

Yeah it is. But it needs more thinking. Let me close this for now.

@rullzer rullzer closed this Mar 7, 2018
@rullzer rullzer deleted the publicshare-general branch March 7, 2018 15:11
@MorrisJobke MorrisJobke removed this from the Nextcloud 14 milestone Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants