Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

[expo-cli] upload:android - add better error messages for issues with downloading archive file #2384

Merged
merged 4 commits into from
Jul 23, 2020

Conversation

dsokal
Copy link
Contributor

@dsokal dsokal commented Jul 22, 2020

Follow-up PR to https://github.com/expo/turtle-v2/pull/258.

I've noticed that we're not providing good error messages when we failed to download the archive file.
There are two main reasons for this to happen:

  • the user provided an incorrect URL (404 error)
  • the download URL has expired (403 error from AWS S3).

In this PR, I'm adding proper error messages for these issues.

CHANGELOG.md Outdated Show resolved Hide resolved
dsokal and others added 2 commits July 22, 2020 17:02
…/errors.ts

Co-authored-by: Wojciech Kozyra <wojciech.kozyra@swmansion.com>
Copy link
Contributor

@cruzach cruzach left a comment

Choose a reason for hiding this comment

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

🥇

Copy link
Member

@byCedric byCedric left a comment

Choose a reason for hiding this comment

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

👍

@dsokal dsokal merged commit 375ebff into master Jul 23, 2020
@dsokal dsokal deleted the @dsokal/ss-add-better-error-messages branch July 23, 2020 07:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants