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

fix file info delete query #5042

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

M-Evans
Copy link

@M-Evans M-Evans commented Oct 6, 2024

Summary

title.

Ticket Link

introduced in #4746, fixes #4947

Additional Information

I've not touched the bad query in mm auth layer because the schema is too different from the correct schema for the file_info table.

CREATE TABLE file_info (
    id varchar(26) NOT NULL,
    create_at BIGINT NOT NULL,
    delete_at BIGINT,
    name TEXT NOT NULL,
    extension VARCHAR(50) NOT NULL,
    size BIGINT NOT NULL,
    archived BOOLEAN
, path varchar(512));

@mattermost-build
Copy link
Contributor

Hello @M-Evans,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.

Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.

This is a standard procedure for many open source projects.

Please let us know if you have any questions.

We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

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 this pull request may close these issues.

Bug: Using docker deploy, Can't delete attachment.
2 participants