Skip to content

Docker fix #297

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

Closed
wants to merge 29 commits into from
Closed

Docker fix #297

wants to merge 29 commits into from

Conversation

y4my4my4m
Copy link

@y4my4my4m y4my4my4m commented Aug 13, 2024

This is this PR but with the additional fix you mentioned and some others

gmargriff and others added 29 commits April 16, 2024 19:26
Syncing last repo changes
Allow usage of command inputs on baremetal Windows OS while keeping disable for incompatible systems (UNIX + Docker and MacOS)
ALTER TABLE `storage_team`
ADD PRIMARY KEY (`accountId`,`itemId`),
ADD KEY `itemId` (`itemId`);
ADD CONSTRAINT `storage_team_ibfk_1` FOREIGN KEY (`accountId`) REFERENCES `accounts` (`accountId`) ON DELETE CASCADE ON UPDATE CASCADE;
Copy link
Author

Choose a reason for hiding this comment

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

ive modified these two SQL because i was getting errors about it having two primary keys btw

@y4my4my4m
Copy link
Author

seems like i dont have access to resolve the conflicting files...

@exectails
Copy link
Member

Are you certain it's an access issue? I'm getting the message that the conflicts are "too complex to resolve in the web editor", but you should be able to do it locally.

@y4my4my4m
Copy link
Author

@exectails you're right, i'll work on them locally.

@y4my4my4m
Copy link
Author

Those changes were pulled in the other fork. So this PR is the same as PR #256 now. Closing

@y4my4my4m y4my4my4m closed this Sep 2, 2024
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.

3 participants