Title: major refactor by ehanson8 · Pull Request #19 · MITLibraries/dspace-api-python-scripts · GitHub
Open Graph Title: major refactor by ehanson8 · Pull Request #19 · MITLibraries/dspace-api-python-scripts
X Title: major refactor by ehanson8 · Pull Request #19 · MITLibraries/dspace-api-python-scripts
Description: What does this PR do? This is a fundamental refactoring of dsaps that was requested to be reviewed in its entirety. Much of the code that was in cli.py was shifted to workflows.py. metadata.py and helpers.py were created to improve the overall organization of the code. Business logic that was coded into the Client methods was shifted to helpers.py. conftest.py was significantly expanded and actual CSV files were added as fixtures to give developers a better sense of the expected input files. Functions and methods were reorganized in alphabetical order Includes new or updated dependencies? YES
Open Graph Description: What does this PR do? This is a fundamental refactoring of dsaps that was requested to be reviewed in its entirety. Much of the code that was in cli.py was shifted to workflows.py. metadata.py and...
X Description: What does this PR do? This is a fundamental refactoring of dsaps that was requested to be reviewed in its entirety. Much of the code that was in cli.py was shifted to workflows.py. metadata.py and...
Opengraph URL: https://github.com/MITLibraries/dspace-api-python-scripts/pull/19
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:97402860-2cf5-ece2-6533-400d9947d0e5 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | D83E:2C0E24:E513CE:1364D57:6A4D5D82 |
| html-safe-nonce | 32caa8f65c735e80126f0afa8d8c375c0ef72a5410d4222ea7a0b931a74a9b46 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEODNFOjJDMEUyNDpFNTEzQ0U6MTM2NEQ1Nzo2QTRENUQ4MiIsInZpc2l0b3JfaWQiOiIxNjA0MzkzNDE4OTgwNDE2ODk4IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | d616c3fc6c2d1f6407ac4ab8281a0a0adc328276c4a5f8dd96e3a14c8e862b00 |
| hovercard-subject-tag | pull_request:599785965 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/MITLibraries/dspace-api-python-scripts/pull/19/files |
| twitter:image | https://avatars.githubusercontent.com/u/18265491?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/18265491?s=400&v=4 |
| og:image:alt | What does this PR do? This is a fundamental refactoring of dsaps that was requested to be reviewed in its entirety. Much of the code that was in cli.py was shifted to workflows.py. metadata.py and... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 2a1443ff9733a3016431263ea73c93d8fe00c195eb4a7670ac30e7848f045441 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/MITLibraries/dspace-api-python-scripts git https://github.com/MITLibraries/dspace-api-python-scripts.git |
| octolytics-dimension-user_id | 2176469 |
| octolytics-dimension-user_login | MITLibraries |
| octolytics-dimension-repository_id | 170196074 |
| octolytics-dimension-repository_nwo | MITLibraries/dspace-api-python-scripts |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 170196074 |
| octolytics-dimension-repository_network_root_nwo | MITLibraries/dspace-api-python-scripts |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | f3bdd3157ea1f0fd738d408eceeead721acf1dc3 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width