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

Fix uploading datapackages from iTAK iOS client #90

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

Matthew-Beckett
Copy link

This PR fixes the datapackage server to implement the upload endpoint used by the iTAK client for sharing datapackages with the server.

Although this implementation works and uploads the files successfully in a manner which appears to work for Android clients. The iOS client returns the same error as previous and still cannot search for datapackages without returning an error.

This PR at least makes it so that aware iOS users can upload datapackages with the knowledge they did indeed succeed and Android users will be able to download them.

@Matthew-Beckett
Copy link
Author

!depends #23 #89

@Matthew-Beckett
Copy link
Author

!link #81

@tkuester
Copy link
Owner

@Matthew-Beckett Thanks for the pull request!

Would you mind splitting out the docker file commits? I'm not quite ready to bring docker and build pipelines into the project, but would love to bring in your commits and credit you for your work!

Comment on lines +2 to +5
pull_request:
push:
branches:
- 'main'
Copy link

@deemoowoor deemoowoor May 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can change these to:

on:
  workflow_dispatch:

and this workflow will only be activated by manual triggering from the Actions menu

https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_dispatch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants