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

Overhaul structure and functionality #43

Open
wants to merge 83 commits into
base: main
Choose a base branch
from
Open

Overhaul structure and functionality #43

wants to merge 83 commits into from

Conversation

DavisRayM
Copy link
Contributor

Changes

  • Add CRUD classes to help with CRUD operations for the models
  • Re-organize project structure
  • Clean-up Hyperfile lifecycle functions

Commits

  • refactor: Reorganize entire project structure
  • refactor(url): remove extra api/v1 prefix
  • refactor(models): ensure relationship points to actual models
  • refactor(server): use CRUDServer class instead of model
  • lint: run formatter on files
  • refactor: return request and session ID
  • refactor(api-helper): Add OnaDataAPIClient class
  • refactor(auth): Use get_current_user for retrieving authenticated user
  • refactor(configuration): Use CRUD class instead of model
  • refactor(config): Add DOWNLOAD_EXPIRE_SECONDS config
  • refactor(schema): add missing fields to FileCreate schema
  • feat(crud-hyper): add create and get helper functions

@DavisRayM DavisRayM requested a review from KipSigei June 8, 2023 13:23
@DavisRayM
feat(helpers): Add Importer helper class for CSV -> Hyper imports
d6953e3
@DavisRayM
feat(onadata): add functions to help with downloading exports
cabc901
@DavisRayM
refactor(crud-user): Add helper functions to help with upstream syncing
6540e42
@DavisRayM
refactor(main): initialize logging config
09c98a7
@DavisRayM
refactor(schema): Update FileListItem schema
8490751
@DavisRayM
refactor(file-api): decouple and reorganize file routes
0468aca
@DavisRayM
refactor(sync): re-introduce hyperfile syncing functionality
953b579
@DavisRayM
chore: update logger unique ID
f738c19
@DavisRayM
refactor(sync): ensure file syncing works as expected
874974d
@DavisRayM
chore(deprecation): Add DEPRECATED comment on unused files
e493d17
@DavisRayM
chore(unused module): remove unused module
9108f98
@DavisRayM
refactor(url): ensure server URLs are cleaned
ce4f77b
@DavisRayM
refactor(build): update used docker file
8dd65d0
@DavisRayM
chore(tox): ensure sh is allowed
20a355e
@DavisRayM
refactor(test): update default settings
c50b3f7
@DavisRayM
refactor(settings): remove unused settings file
96469dd
@DavisRayM
refactor(config): remove unused configuration
ba360e2
@KipSigei
Fix data types
fb424eb 
Signed-off-by: Kipchirchir Sigei <arapgodsmack@gmail.com>
@KipSigei
The from_orm method is deprecated; use model_validate
e985026 
Signed-off-by: Kipchirchir Sigei <arapgodsmack@gmail.com>
@KipSigei KipSigei force-pushed the authentication-changes branch 4 times, most recently from 88001e5 to 95a58e4 Compare July 23, 2024 07:51
@KipSigei
Update logging configs
b78612c 
Signed-off-by: Kipchirchir Sigei <arapgodsmack@gmail.com>
@KipSigei
Enable debug mode and fix deprecation warnings
04e809d 
Signed-off-by: Kipchirchir Sigei <arapgodsmack@gmail.com>
@KipSigei
Fox pydantic validation errors
8b98bb1 
Signed-off-by: Kipchirchir Sigei <arapgodsmack@gmail.com>
KipSigei and others added 17 commits August 13, 2024 14:37
@KipSigei
Update lifecycle handlers
ead7162 
Signed-off-by: Kipchirchir Sigei <arapgodsmack@gmail.com>
@ukanga
[fix] Black reformatting
ff64342
@ukanga
fix(importer): DataFrame dtypes is of type Series
9fa59f7 
AttributeError: 'Series' object has no attribute 'iteritems'
@ukanga
feat(file): Redirect to download URL when .hyper extension is present
3eba870
@ukanga
feat(file): Add download_url to the file list
6e531cf
@ukanga
fix: Ensure the api prefix url is included in the files url/download_url
6f52e3c
@ukanga
refactor(app/core): move FailedExternalRequest to remove a circular i…
420115d 
…mport
@ukanga
refactor(app): remove onadata api query check from the hyperfile crud…
f459fcb 
… module
@ukanga
chore: Add logging to files endpoint
bf94f6d
@ukanga
refactor: use request.url.scheme
97db8ed
@ukanga
chore: Set MEDIA_ROOT from environment variable
df28bd0
@ukanga
refactor: Use python:3.10 image for docker build
b01d8c4
@ukanga
refactor: expect only integers for form_id and user_id in get_using_form
3184ad3
@ukanga
chore(files): log request headers
1d86249
@ukanga
chore(files): log request scope
2217c80
@ukanga
refactor: pick URL scheme from x-forwarded-proto headers
d7579b1
@ukanga
chore: validate form_id on files endpoint
f0fcc61
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KipSigei KipSigei KipSigei left review comments

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
3 participants
@DavisRayM @KipSigei @ukanga