Skip to content

Check bundle url to make sure it exists before submitting transaction on chain - #97

Merged
jlmonroy13 merged 2 commits into
mainfrom
eng-308/check-bundle-url
Feb 4, 2025
Merged

Check bundle url to make sure it exists before submitting transaction on chain #97
jlmonroy13 merged 2 commits into
mainfrom
eng-308/check-bundle-url

Conversation

@jlmonroy13

Copy link
Copy Markdown
Contributor

@dmanjunath dmanjunath left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nice and simple!

Comment thread src/commands/project/content.ts Outdated
try {
await axios.head(args.URL);
} catch (error) {
this.error(`The URL ${args.URL} does not exist or is not accessible.`);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

publicly accessible*

@jlmonroy13
jlmonroy13 merged commit 6326f44 into main Feb 4, 2025
@jlmonroy13
jlmonroy13 deleted the eng-308/check-bundle-url branch February 4, 2025 01:05
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.

2 participants