Skip to content

feat: 28 Files2vue compatibility - #4666

Merged
max-nextcloud merged 13 commits into
mainfrom
feat/28-files
Aug 27, 2023
Merged

feat: 28 Files2vue compatibility#4666
max-nextcloud merged 13 commits into
mainfrom
feat/28-files

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Aug 11, 2023

Copy link
Copy Markdown
Member

nextcloud/server#39808

Fixes#4626

F2V final overview https://github.com/nextcloud/server/issues/39914

@skjnldsvskjnldsv changed the title feat: 28 files compatibilityfeat: 28 Files headers compatibilityAug 11, 2023
@skjnldsvskjnldsv self-assigned this Aug 11, 2023
Comment threadwebpack.config.js
Comment on lines +52 to +57
webpackConfig.plugins.push(new webpack.ProvidePlugin({
Buffer: ['buffer', 'Buffer'],
}))
webpackConfig.resolve.fallback = {
buffer: require.resolve('buffer'),
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this be upstreamed?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

No clue, but it's required somehow
ref nextcloud-libraries/webpack-vue-config#450 (comment)

@cypress

cypressBot commented Aug 11, 2023

Copy link
Copy Markdown

Passing run #11869 ↗︎

0133180Flakiness0

Details:

feat: 28 Files2vue compatibility
Project: TextCommit:2050c7e439
Status: PassedDuration: 04:08 💡
Started: Aug 27, 2023 7:21 PMEnded: Aug 27, 2023 7:25 PM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@juliusknorr

Copy link
Copy Markdown
Member

Will take over for finishing tests and adjusting cypress to work with the f2v merge

@juliusknorrjuliusknorr changed the title feat: 28 Files headers compatibilityfeat: 28 Files2vue compatibilityAug 18, 2023
Comment threadcypress/e2e/nodes/ImageView.spec.js
Comment threadcypress/e2e/attachments.spec.js Outdated
skjnldsvand others added 8 commits August 27, 2023 10:41
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Max <max@nextcloud.com>
File names are now rendered in two spans
which breaks the assertion that the row contains `test.md`.
But that is nothing text should worry about anyway.
Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud

Copy link
Copy Markdown
Collaborator

/compile

@max-nextcloud

max-nextcloud commented Aug 27, 2023

Copy link
Copy Markdown
Collaborator

cypress tests that failed on the last run succeeded for me locally. 🤷

Failure was introduced later in server master branch.
I'm trying to address it here: nextcloud/server#40065

@max-nextcloud
max-nextcloudforce-pushed the feat/28-files branch 2 times, most recently from 38094e1 to 140143eCompareAugust 27, 2023 14:37
juliusknorrand others added 2 commits August 27, 2023 17:17
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud
max-nextcloudforce-pushed the feat/28-files branch 4 times, most recently from 904ffa8 to 542ddbbCompareAugust 27, 2023 18:53
See also nextcloud/server#40065
Signed-off-by: Max <max@nextcloud.com>
basename and extension are now split.
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud

Copy link
Copy Markdown
Collaborator

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@max-nextcloud

Copy link
Copy Markdown
Collaborator

Follow up ticket:

  • BUG: File list header not shown in a folder. filelist._currentDir is undefined and falls back to /
  • Public share file list still missing (therefore rich workspace and file open tests for that are broken)
  • New file creation still missing. we're not even testing that yet. Add some tests once it's there.
  • remove exception handling for fix(files): do not load legacy files app js server#40065 once that is merged.

@max-nextcloud
max-nextcloud merged commit f85ca3d into mainAug 27, 2023
@max-nextcloud
max-nextcloud deleted the feat/28-files branch August 27, 2023 19:37
@max-nextcloudmax-nextcloud mentioned this pull request Aug 27, 2023
4 tasks
Sign up for freeto 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.

Move to new header API

4 participants

@skjnldsv@juliusknorr@max-nextcloud@nextcloud-command