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

[Bug]: Confusing "Add remote share" dialog popup in Files App #38589

Closed
6 of 9 tasks
SergeyMosin opened this issue Jun 1, 2023 · 1 comment
Closed
6 of 9 tasks

[Bug]: Confusing "Add remote share" dialog popup in Files App #38589

SergeyMosin opened this issue Jun 1, 2023 · 1 comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug feature: federation

Comments

@SergeyMosin
Copy link

⚠️ This issue respects the following points: ⚠️

  • This is a bug, not a question or a configuration/webserver/proxy issue.
  • This issue is not already reported on Github (I've searched it).
  • Nextcloud Server is up to date. See Maintenance and Release Schedule for supported versions.
  • Nextcloud Server is running on 64bit capable CPU, PHP and OS.
  • I agree to follow Nextcloud's Code of Conduct.

Bug description

This (see image) pops up in files App when there are pending shares

I think the "Add remote share" button should say "Cancel" and CSS fixes are needed for the top [x] button

image

Steps to reproduce

  1. Ask someone to send you a Federated Share
  2. Do NOT accept it via "Bell" notification
  3. Got to the Files App or reload the page if you are already there
  4. 💥 the "confusing" dialog pops up

image

Expected behavior

The dialog should Not be confusing

Installation method

Community Docker image

Nextcloud Server version

25

Operating system

Debian/Ubuntu

PHP engine version

PHP 8.0

Web server

Nginx

Database engine version

MySQL

Is this bug present after an update or on a fresh install?

Fresh Nextcloud Server install

Are you using the Nextcloud Server Encryption module?

Encryption is Disabled

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

fresh/standard

List of activated Apps

fresh/standard

Nextcloud Signing status

NA

Nextcloud Logs

NA

Additional info

NA

@SergeyMosin SergeyMosin added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Jun 1, 2023
@danxuliu
Copy link
Member

Fixed in #37697.

Note that the dialog should not have been shown if the Notifications app was enabled, but it was due to a regression. See #44823.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap 25-feedback bug feature: federation
Projects
None yet
Development

No branches or pull requests

4 participants