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

Add cli for tile excel #11463

Open
wants to merge 57 commits into
base: dev/7.6.x
Choose a base branch
from
Open

Add cli for tile excel #11463

wants to merge 57 commits into from

Conversation

chiatt
Copy link
Member

@chiatt chiatt commented Sep 17, 2024

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Description of Change

Adds cli support for BDM

Issues Solved

Re #10476

Checklist

  • I targeted one of these branches:
    • dev/7.6.x (under development): features, bugfixes not covered below
    • dev/7.5.x (main support): regressions, crashing bugs, security issues, major bugs in new features
    • dev/6.2.x (extended support): major security issues, data loss issues
  • I added a changelog in arches/releases
  • I submitted a PR to arches-docs (if appropriate)
  • Unit tests pass locally with my changes
  • I added tests that prove my fix is effective or that my feature works
  • My test fails on the target branch

Accessibility Checklist

Developer Guide

Topic Changed Retested
Color contrast
Form fields
Headings
Links
Keyboard
Responsive Design
HTML validation
Screen reader

Ticket Background

  • Sponsored by:
  • Found by: @
  • Tested by: @
  • Designed by: @

Further comments

@njkim
Run single csv importer from command line, #10476
8f5a2fb
@njkim
Update comment, #10476
4f84161
@njkim
Add user, #10476
30bdffd
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
7225dd9
@njkim
Move to its own command, #10476
7598fd1
@njkim
Implement the branch excel, #10476
5cceebf
@njkim
Remove unused import, 10476
5e9dade
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
f1d8b98
@njkim
Fix the file info, #10476
55b2981
@njkim
Reintroduce the multiprocessing arg, #10476
662780c
@njkim
Pass the multiprocessing, #10476
993c252
@njkim
Pass the quiet param down to single processin, #10476
5eaeb61
@njkim
Save the celery threshold as config, #10476
f350afd
@njkim
Fix the migration order, #10476
ac6465e
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
7edaa58
@njkim
nit, #10376
75fedf1
@njkim
Fix formatting, #10476
7c089fc
@njkim
Update the key name of the files size, #10476
ae755b9
@njkim
Remove unnecessary migration, #10476
739ac2d
@njkim
Fix list of params, #10476
8f9cb6b
@njkim
Add option to add multiprocessing
350e111
@njkim
Optional param, #10476
eb3afbc
@njkim
Add missing return, #10476
93715bf
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
ad3bf3e
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
10008cb
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
7a5cf99
@njkim
Retrun the function instead of response, #10478
e25c61e
@njkim
nit, #10476
a62359f
@njkim
Update read to use InMemoryUploadedFile, #10476
9ef6f37
@njkim
Get file size from os.stat, #10476
f64d0ca
njkim and others added 27 commits August 14, 2024 00:38
@njkim
Fix format, #10476
d0a2bcb
@njkim
Remove unused code, #10476
d8e4a4a
@njkim
Add config to branch excel, #10476
e8a7867
@njkim
Not pass cursor around, create uploadFile object, #10476
84eef9f
@njkim
Not use multiprocessing which closes cursor, #10476
f82e11c
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
d7eb8b5
@njkim
nit confirm admin user, #10476
a6239a2
@njkim
Detailed error message, #10476
c139981
@njkim
Remoev unused read_for_cli function, #10475
1302197
@njkim
Not run celery for cli, #10476
36079bb
@njkim
Fix format, #10476
de38449
@njkim
Add test, #10476
f5668f9
@njkim
Fix format, #10476
f7246d5
@njkim
Add default mode, #10476
9368542
@njkim
Fox formatting, #10476
db7d559
@njkim
Revert the save_to_tile, #10476
718a6a9
@njkim
User DEFAULT_RESOURCE_IMPORT_USER for import, #10476
cfc10a0
@njkim
Merge branch 'dev/7.6.x' into 10476_bulk_data_command_line
9aafee6
@njkim
Add user to save_to_tile function, #10476
d59b407
@njkim
Wrap the file open w/ "with"
52b3241
@njkim
Wrap the file open in the base module, #10476
b94ce54
@njkim
Close file later instead of using with, #10476
f259568
@njkim
Define variables w/ initial value, #10476
04ff74f
@njkim
Handle unexpected errors, #10476
0cde6ac
@njkim
Catch the error message, #10476
57d62a6
@njkim
Fix formatting, #10476
0737cd2
@chiatt
Add support for cli tile excel
fbed4c9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Priority: Medium
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chiatt @njkim