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

adjust method upload collection #24

Merged
merged 4 commits into from
Jan 14, 2025
Merged

adjust method upload collection #24

merged 4 commits into from
Jan 14, 2025

Conversation

denarch22
Copy link
Contributor

🛠️ Changes

Made adjustment to the method of loading the collection so that when it fails it calls the method of removing the blob

📝 Associated issues

#18

adjust method upload
adjust method upload
@denarch22 denarch22 requested review from erikasv and ManuelStardust and removed request for erikasv January 10, 2025 14:09
adjust import
@@ -175,6 +176,7 @@ def upload_collection(self):
Upload the collection and items to the STAC server.
"""
try:

Copy link
Member

Choose a reason for hiding this comment

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

borrar esta línea

@@ -201,8 +203,19 @@ def upload_collection(self):
)

except Exception as e:

Copy link
Member

Choose a reason for hiding this comment

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

borrar esta línea

adjust spaces
@denarch22 denarch22 requested a review from erikasv January 14, 2025 19:27
@denarch22 denarch22 merged commit 765ff80 into main Jan 14, 2025
1 check passed
@denarch22 denarch22 deleted the removeFileOnFailure branch January 14, 2025 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants