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

After upgrading to Nextcloud Deck 1.0.1 the attachments previusly uploaded disappear #1957

Open
floydu opened this issue May 25, 2020 · 11 comments

Comments

@floydu
Copy link

floydu commented May 25, 2020

Describe the bug
Deck Version: 1.0.1
After upgrading to Deck 1.0.1 the card attachments disappear. Only the icon that warns that there are attachments on the card is shown, however when the attachments tab is opened, none appears.

When a new attachment is added to the card, the old, missing attachments appear.

To Reproduce
Steps to reproduce the behavior:

  1. The user opens a card in Deck that has attachments that had been added before the upgrade to version 1.0.1 of Deck. (upgrade from version 0.80)
  2. When you open the attachments tab, none appear. As if something had never been attached before.
  3. When the user attaches a new attachment on that same card,
      the old attachments that had previously disappeared now reappear.
    Expected behavior
    Users can view attachments that they have previously uploaded to Nextcloud Deck.

You can see that the card says have 57 attatcments.
Screenshots
You can see that the card have attachments previusly uploaded
image
When user opens the card, ther is no attachments.
image
When user upload another attachment, the old attachments reapear.
image

Client details:

  • OS: Windows 10
  • Browser Chrome
  • Version Lastest
  • Device: Windows Desktop
Server details

Operating system:

**Web server: Nextclud Deck 18.0.4
List of activated apps:

Enabled:

  • activity: 2.11.0
  • admin_audit: 1.8.0
  • cloud_federation_api: 1.1.0
  • comments: 1.8.0
  • dav: 1.14.0
  • deck: 1.0.1
  • federatedfilesharing: 1.8.0
  • federation: 1.8.0
  • files: 1.13.1
  • files_accesscontrol: 1.8.1
  • files_pdfviewer: 1.7.0
  • files_sharing: 1.10.1
  • files_trashbin: 1.8.0
  • files_versions: 1.11.0
  • firstrunwizard: 2.7.0
  • groupfolders: 6.0.6
  • groupquota: 0.1.2
  • impersonate: 1.5.0
  • logreader: 2.3.0
  • lookup_server_connector: 1.6.0
  • notes: 3.3.0
  • notifications: 2.6.0
  • oauth2: 1.6.0
  • onlyoffice: 4.1.4
  • password_policy: 1.8.0
  • photos: 1.0.0
  • privacy: 1.2.0
  • provisioning_api: 1.8.0
  • quota_warning: 1.7.0
  • recommendations: 0.6.0
  • serverinfo: 1.8.0
  • settings: 1.0.0
  • sharebymail: 1.8.0
  • spreed: 8.0.9
  • support: 1.1.0
  • suspicious_login: 3.1.0
  • systemtags: 1.8.0
  • theming: 1.9.0
  • twofactor_admin: 2.1.0
  • twofactor_backupcodes: 1.7.0
  • twofactor_totp: 4.1.3
  • user_ldap: 1.8.0
  • viewer: 1.2.0
  • workflowengine: 2.0.0

Nextcloud configuration:
{
"system": {
"apps_paths": [
{
"path": "/snap/nextcloud/current/htdocs/apps",
"url": "/apps",
"writable": false
},
{
"path": "/var/snap/nextcloud/current/nextcloud/extra-apps",
"url": "/extra-apps",
"writable": true
}
],
"supportedDatabases": [
"mysql"
],
"memcache.locking": "\OC\Memcache\Redis",
"memcache.local": "\OC\Memcache\Redis",
"redis": {
"host": "REMOVED SENSITIVE VALUE",
"port": 0
},
"passwordsalt": "REMOVED SENSITIVE VALUE",
"secret": "REMOVED SENSITIVE VALUE",
"trusted_domains": [
"localhost",
"REMOVED SENSITIVE VALUE"
"REMOVED SENSITIVE VALUE"
],
"datadirectory": "REMOVED SENSITIVE VALUE",
"dbtype": "mysql",
"version": "18.0.4.2",
"overwrite.cli.url": "REMOVED SENSITIVE VALUE"
"dbname": "REMOVED SENSITIVE VALUE",
"dbhost": "REMOVED SENSITIVE VALUE",
"dbport": "",
"dbtableprefix": "oc_",
"mysql.utf8mb4": true,
"dbuser": "REMOVED SENSITIVE VALUE",
"dbpassword": "REMOVED SENSITIVE VALUE",
"installed": true,
"instanceid": "REMOVED SENSITIVE VALUE",
"twofactor_enforced": "false",
"twofactor_enforced_groups": [
"admin"
],
"twofactor_enforced_excluded_groups": [],
"ldapIgnoreNamingRules": false,
"ldapProviderFactory": "OCA\User_LDAP\LDAPProviderFactory",
"maintenance": false,
"mail_from_address": "REMOVED SENSITIVE VALUE",
"mail_smtpmode": "smtp",
"mail_sendmailmode": "smtp",
"mail_domain": "REMOVED SENSITIVE VALUE",
"mail_smtphost": "REMOVED SENSITIVE VALUE",
"mail_smtpport": "25",
"loglevel": 0
}
}

or

Are you using an external user-backend, if yes which one: LDAP/ActiveDirectory/Webdav/...

ActiveDirectory

@floydu
Copy link
Author

floydu commented May 26, 2020

Does any solution for this issue?

@juliushaertl
Copy link
Member

Please check your browser console if there are any errors as the issue template suggests 😉

@floydu
Copy link
Author

floydu commented May 29, 2020

There is the console warnings on Brave Browser 1.9.72:

Notifications permissions granted
search.js?v=3cd45317-75:67 New search handler registered
snap.js:143 [Violation] Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952
addEvent @ tooltip.js:22
(anonymous) @ tooltip.js:22
listen @ tooltip.js:22
t.exports @ tooltip.js:22
J @ init.js:36
(anonymous) @ main.js:35
c @ jquery.js:564
fireWith @ jquery.js:564
ready @ jquery.js:564
P @ jquery.js:564
snap.js:143 [Violation] Added non-passive event listener to a scroll-blocking 'touchmove' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952
addEvent @ tooltip.js:22
(anonymous) @ tooltip.js:22
listen @ tooltip.js:22
t.exports @ tooltip.js:22
J @ init.js:36
(anonymous) @ main.js:35
c @ jquery.js:564
fireWith @ jquery.js:564
ready @ jquery.js:564
P @ jquery.js:564
snap.js:143 [Violation] Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952
addEvent @ tooltip.js:22
(anonymous) @ tooltip.js:22
listen @ tooltip.js:22
enable @ tooltip.js:22
f @ init.js:36
J @ init.js:36
(anonymous) @ main.js:35
c @ jquery.js:564
fireWith @ jquery.js:564
ready @ jquery.js:564
P @ jquery.js:564
snap.js:143 [Violation] Added non-passive event listener to a scroll-blocking 'touchmove' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952
addEvent @ tooltip.js:22
(anonymous) @ tooltip.js:22
listen @ tooltip.js:22
enable @ tooltip.js:22
f @ init.js:36
J @ init.js:36
(anonymous) @ main.js:35
c @ jquery.js:564
fireWith @ jquery.js:564
ready @ jquery.js:564
P @ jquery.js:564
(index):1 [DOM] Password forms should have (optionally hidden) username fields for accessibility: (More info: https://goo.gl/9p2vKq) <form id=​"sudo-login-form" class=​"hidden">​…​​
[Violation] Forced reflow while executing JavaScript took 110ms


@floydu
Copy link
Author

floydu commented Jun 5, 2020

Does any solution for this issue?

@pschopen
Copy link
Contributor

pschopen commented Jun 8, 2020

Refreshing the page helps in my case. So after opening the card's sidebar press cmd+r (on Mac) and the attachments will show up. Tested in Chrome on Mac.

@floydu
Copy link
Author

floydu commented Jun 19, 2020

There is any solution por this issue?

@reox
Copy link
Contributor

reox commented Jun 22, 2020

I have the same issue here with 1.0.3 and noticed this also in 1.0.2.

In all cases the attachments are not displayed when switching cards. Reloading the page helps in all cases.

Steps to reproduce:

  • Select an initial card (attachments are displayed)
  • Select another card which has attachments -> attachments are not displayed
  • Reload page -> attachments are displayed again

I do not see any errors on the JS console, however there are always these warnings:

JQMIGRATE: Migrate is installed, version 1.4.1 jquery-migrate.min.js:2:551
This page uses the non standard property “zoom”. Consider using calc() in the relevant property values, or using “transform” along with “transform-origin: 0 0”. addStylesClient.js:220:17
Proxying an event bus of version 1.1.2 with 0.2.1 ProxyBus.js:65:14
Handlebars is deprecated: please ship your own, this will be removed in Nextcloud 20 globals.js:66:15
Notifications permissions denied App.vue:248
New search handler registered search.js:67:12
moment is deprecated: please ship your own, this will be removed in Nextcloud 20 globals.js:66:15
The escapeHTML library is deprecated! It will be removed in nextcloud 19. 3 globals.js:66:15

@Firerazer
Copy link

I also observe this kind of issue for projects with a folder linked. The same steps as described in previous messages allow to reproduce, so I do not copy/paste the steps

@juliushaertl
Copy link
Member

I can reproduce this now, it seems that the attachments are not fetched at all when switching between the cards without closing the sidebar in between.

@olegbolden
Copy link

olegbolden commented Nov 17, 2020

Is there any progress on resolving this issue?
Today I was informed of the next major update 1.2, but this issue is not mentioned among bugfixes.
We can not use Deck for more than a month due to this issue with our partners.

@kapitan-iglu
Copy link

Hi,
I can confirm that this issue still exists on NextCloud v18.0.12 with Deck v1.2.3.

If I browse cards only using "Open in bigger view", then I had no problem with attachment visibility.

If I use side panel to view card details then attachment list is sometimes empty even if there are attachment and icon on card shows right attachment count.
When side panel is shown with empty attachment list this helps me to recover:

  1. Reload page (Ctrl+R) and switch back to Attachments tab (as suggested by reox).
  2. Before moving to another card with opened side panel, close it first (as suggested by juliushaertl, really annoying :).
  3. Click "Open in bigger view" and move to Attachments tab.
  4. Do not use side panel, use "bigger view" instead :)

Each time I click on new card with side bar shown, these two errors appears in browser console:

xhr.js:177: PROPPATCH https://.../nextcloud/remote.php/dav/comments/deckCard/17 404 (Not Found)
xhr.js:177
PROPPATCH https://.../nextcloud/remote.php/dav/comments/deckCard/17 404 (Not Found)
(anonymous)	@	popper.js:25
t.exports	@	popper.js:25
t.exports	@	main.js?v=89372a2d-0:1665
Promise.then (async)		
c.request	@	main.js?v=89372a2d-0:1665
(anonymous)	@	popper.js:25
(anonymous)	@	CommentApi.js:26
c	@	popper.js:25
(anonymous)	@	popper.js:25
forEach.e.<computed>	@	popper.js:25
B	@	main.js?v=89372a2d-0:1560
a	@	main.js?v=89372a2d-0:1560
(anonymous)	@	main.js?v=89372a2d-0:1560
(anonymous)	@	main.js?v=89372a2d-0:1560
(anonymous)	@	CommentApi.js:26
(anonymous)	@	CommentApi.js:26
c	@	popper.js:25
(anonymous)	@	popper.js:25
forEach.e.<computed>	@	popper.js:25
L	@	main.js?v=89372a2d-0:1560
a	@	main.js?v=89372a2d-0:1560
(anonymous)	@	main.js?v=89372a2d-0:1560
(anonymous)	@	main.js?v=89372a2d-0:1560
markCommentsAsRead	@	CommentApi.js:26
(anonymous)	@	vue.runtime.esm.js:8
d.dispatch	@	vuex.esm.js:6
dispatch	@	vue.runtime.esm.js:8
(anonymous)	@	main.js?v=89372a2d-0:2182
c	@	popper.js:25
(anonymous)	@	popper.js:25
forEach.e.<computed>	@	popper.js:25
Do	@	main.js?v=89372a2d-0:2182
a	@	main.js?v=89372a2d-0:2182
Promise.then (async)		
Do	@	main.js?v=89372a2d-0:2182
a	@	main.js?v=89372a2d-0:2182
(anonymous)	@	main.js?v=89372a2d-0:2182
(anonymous)	@	main.js?v=89372a2d-0:2182
loadComments	@	main.js?v=89372a2d-0:2182
handler	@	main.js?v=89372a2d-0:2182
mn.run	@	vue.runtime.esm.js:8
un	@	vue.runtime.esm.js:8
(anonymous)	@	vue.runtime.esm.js:8
Zt	@	vue.runtime.esm.js:8
Promise.then (async)		
Wt	@	vue.runtime.esm.js:8
ee	@	vue.runtime.esm.js:8
(anonymous)	@	vue.runtime.esm.js:8
mn.update	@	vue.runtime.esm.js:8
ut.notify	@	vue.runtime.esm.js:8
set	@	vue.runtime.esm.js:8
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
Pt.updateRoute	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
yt	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
f	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
yt	@	main.js?v=89372a2d-0:2161
Pt.confirmTransition	@	main.js?v=89372a2d-0:2161
Pt.transitionTo	@	main.js?v=89372a2d-0:2161
e.push	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
Ht.push	@	main.js?v=89372a2d-0:2161
openCard	@	main.js?v=89372a2d-0:2182
Rt	@	vue.runtime.esm.js:8
n	@	vue.runtime.esm.js:8
i._wrapper	@	vue.runtime.esm.js:8

and:

createError.js:16: Uncaught (in promise) Error: Request failed with status code 404
createError.js:16
Uncaught (in promise) Error: Request failed with status code 404
    at t.exports (createError.js:16)
    at t.exports (settle.js:17)
    at XMLHttpRequest.p.onreadystatechange (xhr.js:62)
t.exports	@	popper.js:25
t.exports	@	main.js?v=89372a2d-0:1665
p.onreadystatechange	@	popper.js:25
Promise.then (async)		
Do	@	main.js?v=89372a2d-0:2182
a	@	main.js?v=89372a2d-0:2182
Promise.then (async)		
Do	@	main.js?v=89372a2d-0:2182
a	@	main.js?v=89372a2d-0:2182
(anonymous)	@	main.js?v=89372a2d-0:2182
(anonymous)	@	main.js?v=89372a2d-0:2182
loadComments	@	main.js?v=89372a2d-0:2182
handler	@	main.js?v=89372a2d-0:2182
mn.run	@	vue.runtime.esm.js:8
un	@	vue.runtime.esm.js:8
(anonymous)	@	vue.runtime.esm.js:8
Zt	@	vue.runtime.esm.js:8
Promise.then (async)		
Wt	@	vue.runtime.esm.js:8
ee	@	vue.runtime.esm.js:8
(anonymous)	@	vue.runtime.esm.js:8
mn.update	@	vue.runtime.esm.js:8
ut.notify	@	vue.runtime.esm.js:8
set	@	vue.runtime.esm.js:8
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
Pt.updateRoute	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
yt	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
f	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
r	@	main.js?v=89372a2d-0:2161
yt	@	main.js?v=89372a2d-0:2161
Pt.confirmTransition	@	main.js?v=89372a2d-0:2161
Pt.transitionTo	@	main.js?v=89372a2d-0:2161
e.push	@	main.js?v=89372a2d-0:2161
(anonymous)	@	main.js?v=89372a2d-0:2161
Ht.push	@	main.js?v=89372a2d-0:2161
openCard	@	main.js?v=89372a2d-0:2182
Rt	@	vue.runtime.esm.js:8
n	@	vue.runtime.esm.js:8
i._wrapper	@	vue.runtime.esm.js:8

Browser:

  • Vivaldi 3.5.2115.87 + JavaScript V8 8.7.220.31
  • Chromium 87.0.4280.141 + JavaScript V8 8.7.220.31

Let me know to provide more info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants