Skip to content

[18.0][IMP] dms + dms_file: Upload file and create dms.file directly#472

Open
victoralmau wants to merge 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-dms-upload-file
Open

[18.0][IMP] dms + dms_file: Upload file and create dms.file directly#472
victoralmau wants to merge 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-dms-upload-file

Conversation

@victoralmau
Copy link
Copy Markdown
Member

@victoralmau victoralmau commented Apr 29, 2026

Upload file and create dms.file directly

Related to #470 (comment)

Please @CarlosRoca13 can you review it?

@Tecnativa

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @CarlosRoca13,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added mod:dms Module dms mod:dms_field Module dms_field series:18.0 labels Apr 29, 2026
@victoralmau victoralmau changed the title [18.0][WIP] dms: Upload file and create dms.file directly [18.0][WIP] dms + dms_file: Upload file and create dms.file directly Apr 29, 2026
@pedrobaeza pedrobaeza added this to the 18.0 milestone Apr 29, 2026
@victoralmau victoralmau force-pushed the 18.0-imp-dms-upload-file branch from ed13767 to 49d963a Compare April 29, 2026 14:05
Copy link
Copy Markdown
Member

@tarteo tarteo left a comment

Choose a reason for hiding this comment

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

LGTM like what we discussed in my PR. Can't test it functionality at the moment

@pedrobaeza
Copy link
Copy Markdown
Member

@victoralmau still draft?

@victoralmau victoralmau marked this pull request as ready for review May 13, 2026 08:04
@victoralmau victoralmau changed the title [18.0][WIP] dms + dms_file: Upload file and create dms.file directly [18.0][IMP] dms + dms_file: Upload file and create dms.file directly May 13, 2026
@victoralmau
Copy link
Copy Markdown
Member Author

Sorry, it's ready to review

Ping @CarlosRoca13

@pedrobaeza
Copy link
Copy Markdown
Member

Testing this in runboat, the file uploads, but the thumbnail for images is not created. Can you please check?

@victoralmau victoralmau force-pushed the 18.0-imp-dms-upload-file branch from 49d963a to 1ff5f69 Compare May 14, 2026 08:48
@victoralmau
Copy link
Copy Markdown
Member Author

Testing this in runboat, the file uploads, but the thumbnail for images is not created. Can you please check?

Fixed, can you review again?

Comment thread dms/static/src/js/views/dms_file_upload.esm.js Outdated
Comment thread dms/controllers/main.py
@victoralmau victoralmau force-pushed the 18.0-imp-dms-upload-file branch 2 times, most recently from 1ff5f69 to eba492d Compare May 14, 2026 09:52
@victoralmau victoralmau force-pushed the 18.0-imp-dms-upload-file branch from eba492d to 33cc542 Compare May 15, 2026 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:dms_field Module dms_field mod:dms Module dms series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants