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

improve-public-share-interface #38712

Merged
merged 1 commit into from
May 19, 2021
Merged

Conversation

AlexAndBear
Copy link

@AlexAndBear AlexAndBear commented May 7, 2021

Description

This adjustment provides a new way to add a public share to ownCloud.
A new button will be displayed where you can easily add the share to the current
server without entering the string, this requires less user input.
Next to this button, a dropdown menu will be displayed,
where you can see the current server or change the server.
Hitting change a server will now display a prompt with a detailed explanation
and more space to enter another server address.

Related Issue

Tested on

  • Firefox
  • Chrome
  • IE11

Screenshots (if appropriate):

Before:

image

image

After:

image

image

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@AlexAndBear AlexAndBear self-assigned this May 7, 2021
@AlexAndBear
Copy link
Author

AlexAndBear commented May 8, 2021

TODO:

  • Create Docs issue if needed
  • Remove old code
  • Fix remote without http/https
  • Reintroduce loading spinner
  • Fix tests
  • Add changelog item
  • Fix Firefox UI Glitch

@AlexAndBear AlexAndBear force-pushed the improve-public-share-interface branch from 2c6ce78 to 862cfee Compare May 9, 2021 10:49
@AlexAndBear
Copy link
Author

@JammingBen pre-review is welcome 🐰

@AlexAndBear AlexAndBear force-pushed the improve-public-share-interface branch 2 times, most recently from e093bb5 to 0e8e065 Compare May 11, 2021 07:37
@AlexAndBear
Copy link
Author

@phil-davis Patrick told me that, you are a quite good text reviewer, can you look at the texts for the oc.dialogs.drop maybe :) ?

@AlexAndBear AlexAndBear force-pushed the improve-public-share-interface branch from d67cf77 to f2cfe49 Compare May 11, 2021 08:49
@AlexAndBear AlexAndBear marked this pull request as ready for review May 11, 2021 11:38
@AlexAndBear AlexAndBear force-pushed the improve-public-share-interface branch from 7a2e827 to db15d7e Compare May 11, 2021 11:38
@owncloud owncloud deleted a comment from update-docs bot May 11, 2021
@AlexAndBear AlexAndBear requested a review from IljaN May 18, 2021 20:45
Copy link
Member

@VicDeo VicDeo left a comment

Choose a reason for hiding this comment

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

Cool improvement but it needs to respect the setups where OC is located in the subfolder of the server webroot:

  1. Install OC into localhost/owncloud
  2. Create a public folder share
  3. Open it as a guest user
  4. Click 'Add to localhost'

Expected

login screen

Actual

HTTP 404

as it requests
http://localhost/index.php/apps/files# instead of
http://localhost/owncloud/index.php/apps/files#

apps/federatedfilesharing/js/public.js Outdated Show resolved Hide resolved
@AlexAndBear AlexAndBear force-pushed the improve-public-share-interface branch from db15d7e to 52aed0b Compare May 19, 2021 09:50
@AlexAndBear
Copy link
Author

AlexAndBear commented May 19, 2021

@VicDeo good catch, thx!

can you try again?

@AlexAndBear AlexAndBear requested a review from VicDeo May 19, 2021 09:54
.htaccess Outdated Show resolved Hide resolved
@AlexAndBear AlexAndBear force-pushed the improve-public-share-interface branch from d617745 to be707c2 Compare May 19, 2021 12:07
@AlexAndBear AlexAndBear requested a review from VicDeo May 19, 2021 12:07
@sonarcloud
Copy link

sonarcloud bot commented May 19, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

None yet

3 participants