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

Added uploads to components #12057

Merged
merged 7 commits into from
Nov 2, 2022
Merged

Conversation

snipe
Copy link
Owner

@snipe snipe commented Nov 2, 2022

This is almost ready to go. It does the same thing as the addition of models/license file uploads does. I'm just running into a funky layout bug that I need to fix.

I'm halfway through adding uploads for accessories and components. (Ignore the dumb branch name here - it's consumables not components.)

Eventually I'd like to abstract out of some of this stuff, since there is a lot of repeated code. I'm thinking we can turn CanUploadFiles as a trait or something.

@snipe snipe changed the title WIP: Added uploads to components Added uploads to components Nov 2, 2022
@snipe snipe marked this pull request as ready for review November 2, 2022 02:07
@snipe snipe merged commit 1d47d9e into develop Nov 2, 2022
@snipe snipe deleted the features/add_uploads_to_components branch November 2, 2022 02:07
Sign up for free to 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.

1 participant