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.

Sign up for GitHub

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

Jump to bottom

Refactor to support mass insert/update #210

Merged
merged 7 commits into from
Dec 4, 2024
Merged

Refactor to support mass insert/update #210

merged 7 commits into from
Dec 4, 2024

Conversation

maxachis
Copy link

@maxachis maxachis commented Dec 4, 2024 •
edited
Loading

Fixes

Description

  • Update _get_source_getting_function
  • Refactor error checking logic
  • Remove unused models
  • Add file source mapping to SourceMappingEnum
  • Add MAX_CONTENT_LENGTH to Flask App Configuration
  • Add TODO comment

Testing

  • Run tests in /tests to confirm functionality

Performance

  • Impact marginal

Docs

  • No documentation impact

Breaking Changes

  • No breaking changes

@maxachis maxachis changed the title Mc 477 resource mass imports Refactor to support mass insert/update Dec 4, 2024
@maxachis maxachis merged commit 614d632 into dev Dec 4, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@josh-chamberlain josh-chamberlain Awaiting requested review from josh-chamberlain josh-chamberlain is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@maxachis