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

Release 5.0.0 #6248

Closed
24 of 25 tasks
pascalwengerter opened this issue Jan 13, 2022 · 17 comments · Fixed by #6424
Closed
24 of 25 tasks

Release 5.0.0 #6248

pascalwengerter opened this issue Jan 13, 2022 · 17 comments · Fixed by #6424

Comments

@pascalwengerter
Copy link
Contributor

pascalwengerter commented Jan 13, 2022

Waiting for

If time permits

Small optional fixes / wishes (maybe on release branch)

Release checklist

  • pin release issue @pascalwengerter
  • prepare release rc: @kulmann
    • create a branch release-5.0.0 @kulmann
    • update the version in packages/web-integration-oc10/appinfo/info.xml @kulmann
    • update the version in package.json @kulmann
    • update the version in sonar-project.properties @kulmann
    • move changelog/unreleased/* to changelog/VERSION_YYYY-MM-DD @kulmann
    • update the ocis commit id in .drone.env to the most recent one from ocis master (let's see how it goes) @kulmann
    • commit, push and create draft PR (#...) for this branch @kulmann
  • create pre release tag v5.0.0-rc from the release branch @kulmann
  • smoke test
  • if needed, fix issues or write down known issues
  • update YYYY-MM-DD to current date in changelog/VERSION_YYYY-MM-DD
  • Make sure translations are up to date by updating them in Transifex and restarting translation sync job https://drone.owncloud.com/owncloud/translation-sync/ @pascalwengerter
  • repeat above steps if a new rc is needed
  • Run translations sync job once again
  • go to the releases section and click Draft a new Release. Use v5.0.0 as a tag (the v prefix is important) and 5.0.0 as name and publish it
  • ping in internal #btr channel to create a signed release and ask them to upload it to a) the github release page, b) marketplace.owncloud.com, c) portal.owncloud.com, d) customer.owncloud.com
  • merge release branch https://github.com/owncloud/web/pull/... back to master (CI must be green)
@pascalwengerter pascalwengerter pinned this issue Jan 13, 2022
@dschmidt
Copy link
Member

I would really like to see #6156 getting into the release. I made sure to make it readytoreview before my vacation because file/folder handling inconsistencies in different apps have superannoying consequences like certain apps only working in certain contexts... and we need this pr to properly fix it for all apps

@kulmann
Copy link
Member

kulmann commented Feb 10, 2022

There's a regression on current master that prevents public links with password to be loaded from ownCloud 10 backends. Currently investigating...

Propfind for a public link with password gives the following error message:

<?xml version="1.0"?>
<d:multistatus xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns" xmlns:oc="http://owncloud.org/ns"<?xml version="1.0" encoding="utf-8"?>
<d:error xmlns:d="DAV:" xmlns:s="http://sabredav.org/ns">
  <s:exception>Sabre\DAVACL\Exception\NeedPrivileges</s:exception>
  <s:message>User did not have the required privileges ({DAV:}read) for path "public-files/2iYEtDt3MhwfcoI"</s:message>
  <d:need-privileges>
    <d:resource>
      <d:href>/remote.php/dav/public-files/2iYEtDt3MhwfcoI</d:href>
      <d:privilege>
        <d:read/>
      </d:privilege>
    </d:resource>
  </d:need-privileges>
</d:error>

for a request URL like this: http://host.docker.internal:8080/remote.php/dav/public-files/2iYEtDt3MhwfcoI

@dschmidt
Copy link
Member

owncloud/core#39707

@kulmann kulmann mentioned this issue Feb 11, 2022
@pascalwengerter
Copy link
Contributor Author

pascalwengerter commented Feb 11, 2022

Smoke testing oCIS 1.16 with the WOPI deployment example (edge has broken the WOPI integration AFAIK) and the v5.0.0 leads to certification errors (we used to prevent them by visiting the editor URL, e.g. codimd.owncloud.test, and choosing the "accept the risk and continue" button before opening the editor, which doesn't seem to work anymore) false alarm, works as expected. No clue what caused the hickup (perhaps hadn't fully started the necessary services yet?)

@ScharfViktor
Copy link
Contributor

ScharfViktor commented Feb 11, 2022

SmokeTest against ocis

branch: v5.0.0-rc

Case 1: admin share folder(with file) to marie:

Admin:

  • upload big file 30gb - very slow
  • upload big file 10gb - very slow
  • upload folder with 848 files.
  • Check pagination, count items per page.
    ❌ - [ ] upload folder(drag folder) "folder_to_share" with file.pdf Bug: Upload folder (drag and drop) doesn't work #6426
  • share folder "folder_to_share" to marie with edit right

Marie:

  • accept "folder_to_share"
    ❌ - when you try to rename file before accept you get 404 Error. Which I think is right. Do we need button "rename"?
    image

  • rename "folder_to_share" (?admin don't see)

  • upload new file "file.txt" to "folder_to_share" (admin see new file)

  • open pdf file

  • rename file.pdf (use rigth click) (admin see that file changed name)

  • try to move "folder_to_share" to "/" (501 not implement)

  • copy "folder_to_share" to "/"

Admin:

  • add new file.jpeg to "folder_to_share" (marie can see file without access)
  • add file format drawio(marie can open in Draw.io and can edit)
  • move file.pdf to "/" (marie can not see file.pdf)
  • add new "file.txt" create new version. (marie can see new version)
  • open txt file in markdown-editor. Change and save.
    Marie:
  • can download both versions (!error 501)
  • restore old version

Admin:

  • change share "folder_to_share" from "edit" to "viewer" (marie cannot rename/move/delete/download file in folder and cannot create new document in the folder)

Marie:

  • decline folder

Admin:

  • delete folder "folder_to_share" (marie: folder Shares empty)

Case 2: admin share file to marie:

Admin:

  • upload file.jpeg
  • share file.jpeg to marie with rigth viewer (use main menu. find marie by email marie@example.org)

Marie:

  • accept file (marie can open/download/copy file but cannot delete/move/rename)
  • copy file to "/"
  • open file in Mediaviewer
  • download file

Admin:

  • change share "file.jpeg" from "viewer" to "edit"(marie can open/download/copy/rename file)

Marie:

  • rename file (admin don't see)

Admin:

  • unshare "file.jpeg" (marie cannot see "file.jpeg" in Shares/Shared for me but see the copied file)

Case 3: admin create public links to folder

Admin:

  • create folder "for upload"
  • add link with name "to_upload" with role "Uploader" with pass and expiration date
  • open link (can only upload)
  • upload file (admin can see new file)
  • refresh page(page is empty)
    ❌ - [ ] change share "for upload" from "Uploader" to "edit" Bug: Public link. Change role doesn't work #6429
  • open old link (it work as upload)
  • open new link (can open/download/copy/rename/move files)
  • admin add new file to "for upload"
  • refresh link (file exist)
  • admin change pass (refresh link)

Case 4: admin create link to file

Admin:

  • create file png

  • add link with main menu

  • open link(can open, download, see) in the incognito tab

  • change link, add pass

  • refresh page (input pass)

  • delete link

  • refresh page (resource not found)

  • delete file

  • restore

  • delete

  • Empty trash

Case 5: create new user:

Admin:

  • create new account "tom"

Tom:
❌ - [ ] go to account #6430

  • change language

Admin:

  • block user (Login Error access denied)
  • delete user (Logon failed)
  • logout

Findings:

@kulmann
Copy link
Member

kulmann commented Feb 11, 2022

Smoke test v5.0.0-rc against oc10 app

  • ✔️ as gunter - upload arch/mips folder from linux kernel (1945 files)
    - ❌ drag and drop folder upload completely broken 👎 error the server responded with a status of 403 (Forbidden)
    - via folder upload button works

    • progress bar is not sticky
    • expanding the upload progress section slows down browser & upload time significantly
  • ✔️ as gunter - share the folder to alice as viewer
    - sharing is not possible before the upload finished (user invite field doesn't stay focussed)
    - ❌ for reaching the invite form field one needs to tab through the whole file list until the sidebar is reached (focus should be set to sidebar or sidebar panel instead, depending on a11y needs)

  • ✔️ as alice - share is already accepted. declining and accepting again works as expected (buttons in shared with me table, batch action, right sidebar)

  • ✔️ as alice - try do delete file from shared folder
    - individual delete action not visible, good
    - batch delete action not visible, good
    - context menu action not visible, good

  • ✔️ as alice - try to add file
    - not possible (which is expected) but drag & drop upload didn't handle it. opened file in new tab instead.

  • ❌ as alice - copy 16 folders from share to home

    • no visible feedback that it's in progress
    • UI still weirdly usable, ended up in an active upload drop zone. Needed a page reload to be usable again.
    • folder listing after copy shows ? as folder size on some of the copied folders.
  • ✔️ as gunter - add new file to share

  • ✔️ as alice - receive new file in existing share

  • ✔️ as gunter - unshare folder from shared with others

  • ✔️ as alice - try to revisit old share

    • shows resource not found upon page reload, good
    • going back to All files or Shared with me doesn't list the share anymore, good
  • ✔️ as gunter - share folder with alice again, but as editor
    - setting and removing an expiration date work as expected, good

  • ✔️ as alice - rename file

    • via right sidebar, works, good
    • via context menu, works, good
  • ✔️ as gunter - add file

  • ✔️ as alice - delete file

    • via right sidebar, works, good
    • via context menu, works, good
    • via batch action, works, good
  • ✔️ as alice - create public link on any folder

  • ✔️ as alice delete file and recover it

  • ✔️ as alice create version of textfile

  • ✔️ as alice view version history of text file

  • ✔️ as alice download newest version of text file

  • ✔️ download older version of text file

  • ✔️ restore older version of text file

    • no visible feedback about the success
    • preview is not updated
  • ✔️ as alice - upload avatar in classic UI

    • see own avatar in top bar (far right)
    • login as gunter, having a share with alice
      • shared with others page loads and shows avatars
    • avatar in top bar doesn't have correct aspect ratio, see screenshot

Screenshot 2022-02-11 at 15 51 55

  • ✔️ as alice, move a file into subfolder via drag and drop
  • ✔️ as alice, move multiple files & folders into subfolder via drag and drop
  • ✔️ as alice, navigate into a folder, open right sidebar without selection, share current folder with gunter

Findings from old smoke testing:

  • ❌ when a share fails, the submit button of the form stays in the disabled state with spinner. Needs to reset itself instead. (not resolved yet!)

Findings from new smoke testing:

  • ❌ avatar in top bar doesn't have correct aspect ratio, see screenshot (solved)

Screenshot 2022-02-11 at 15 51 55

  • ❌ drag and drop folder upload completely broken (solved)

  • ❌ (error) messages in modal are centered instead of left aligned (introduced by me in views/PublicLink.vue), see screenshot (solved)

Screenshot 2022-02-11 at 16 50 49

@ChrisEdS
Copy link

Settings menu placement looks wrong:

image

Reduce at least the spacing to the gear icon?

@pascalwengerter
Copy link
Contributor Author

Settings menu placement looks wrong:

image

Reduce at least the spacing to the gear icon?

@kulmann that was introduced by giving the .tippy-content additional padding to make the shadow visible 🥴

@pascalwengerter
Copy link
Contributor Author

pascalwengerter commented Feb 11, 2022

I have a fix for the avatar image width/height mess-up. Push straight to release branch? Pushed to the release branch

@kulmann
Copy link
Member

kulmann commented Feb 11, 2022

I have a fix for the avatar image width/height mess-up. Push straight to release branch?

Yep, push it :-)

@ChrisEdS
Copy link

Low contrast in folder action menu

image

@ChrisEdS

This comment was marked as off-topic.

@ScharfViktor

This comment was marked as outdated.

@kulmann

This comment was marked as outdated.

@kulmann
Copy link
Member

kulmann commented Feb 14, 2022

Dark mode search preview has the same background color and parent-folder link color, see screenshot

Screenshot 2022-02-14 at 16 08 04

@pascalwengerter
Copy link
Contributor Author

SmokeTest against oCIS (binary @ current master)

branch: release-5.0.0

Case 1: admin share folder(with file) to marie:

einstein:

  • upload big file 30gb - very slow
  • upload big file 6.6gb - reasonably fast
  • upload folder with 848 files.
  • Check pagination, count items per page.
  • upload folder via drag&drop (7.111 files) - progress bar doesn't move while count goes from 1 to 7.111, then does track progress while counting down again. Upload interupted by OIDC refresh
  • share folder "101_archive to marie with edit right

Marie:

  • accept "101_archive"
  • ❌ when trying to rename file before accept you get 404 Error. => we should not show the rename action on pending shares (folders)
  • rename "101_archive" to "archive_2" works for Marie, but doesn't show renamed folder for einstein
  • upload new file "file.txt" to "101_archive", is visible for einstein
  • open pdf file
  • rename file in "archive_2", einstein sees that file has been renamed
  • try to move "101_archive" to "user home" (leads to error message)
  • copy "101_archive" to "user home" (no visible feedback for copying this many files, browser session is totally wrecked after trying to navigate into copied folder which has only partly copied files)

Einstein:

  • add new file.jpeg to "101_archive" (marie can see file)
  • add file format drawio (marie can open in Draw.io and can edit)
  • move file.pdf to "/" (marie can not see file.pdf)
  • add new "file.txt" create new version. (marie can see new version)
  • open txt file in markdown-editor. Change and save.

Marie:

  • ❌ can download both versions (!error 501)
  • ❌ restore old version toggles between two older states

Einstein:

  • change share "101_archive" from "edit" to "viewer" (marie cannot rename/move/delete/download file in folder and cannot create new document in the folder)

Marie:

  • decline folder

Einstein:

  • delete folder "folder_to_share" (marie: folder Shares empty)

Case 2: admin share file to marie:

Einstein:

  • upload file.jpeg
  • share file.jpeg to marie with viewer permissions

Marie:

  • accept file (marie can open/download/copy file but cannot delete/move/rename)
  • copy file to "/"
  • open file in Mediaviewer
  • download file

Einstein:

  • rename "file.jpeg" to "file2.jpeg", marie doesn't see renamed share but original filename "file.jpeg"
  • change share permissions for "file2.jpeg" from "viewer" to "edit"(marie can open/download/copy/rename file)

Marie:

  • rename file to "file3.jpg", einstein doesn't see renamed share but original filename "file2.jpeg"

Einstein:

  • unshare "file.jpeg" (marie cannot see "file.jpeg" in Shares/Shared for me but see the copied file)

Case 3: einstein create public links to folder

einstein:

  • create folder "for upload"
  • add link with name "to_upload" with role "Uploader" with password and expiration date
  • open link (can only upload)
  • upload file (einstein can see new file)
  • refresh page(page is empty)
  • change share "for upload" from "Uploader" to "edit"
  • open link again (can open/download/copy/rename/move files)
  • einstein add new file to "for upload"
  • refresh link (file exist)
  • einstein change password (refresh link)

Case 4: einstein create link to file

einstein:

  • create file png
  • create view permission link share in sidebar
  • open link in the incognito tab
  • einstein changes link by adding a password
  • incognito refresh page, inputs password, sees file
  • einstein deletes link
  • incognito refresh page, sees only "resource not found"
  • einstein deletes file
  • einstein restores file
  • einstein deletes file
  • einstein empties trash

Case 5: create new user:

Admin:

  • ❌ create new account "tom"

Tom:

Admin:

  • ❌ block user (Login Error access denied)
  • ❌ delete user (Logon failed)
  • ❌ logout

Findings:

@ChrisEdS
Copy link

Jumping file list content in small resolution

Screen.Recording.2022-02-14.at.17.16.49.mov

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 a pull request may close this issue.

5 participants