Skip to content

Add status to outputs#147

Merged
fjogeleit merged 3 commits intofjogeleit:mainfrom
jerturowetz:patch-1
May 4, 2024
Merged

Add status to outputs#147
fjogeleit merged 3 commits intofjogeleit:mainfrom
jerturowetz:patch-1

Conversation

@jerturowetz
Copy link
Copy Markdown

@jerturowetz jerturowetz commented Apr 24, 2024

Closes #146

Hopefully this is the only thing I'd need to change. Not super familiar with Axios here :P

@jerturowetz jerturowetz changed the title Add status to Axios outputs Add status to outputs Apr 24, 2024
@fjogeleit
Copy link
Copy Markdown
Owner

fjogeleit commented Apr 25, 2024

You need to build the action, otherwise your change is not active.

npm run build

also please add the output to the readme response section

@jerturowetz
Copy link
Copy Markdown
Author

jerturowetz commented May 3, 2024

You need to build the action, otherwise your change is not active.

npm run build

also please add the output to the readme response section

@fjogeleit Thanks for the guidance!

@fjogeleit fjogeleit merged commit 7d4a18a into fjogeleit:main May 4, 2024
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.

Any way to retrieve the response status code?

2 participants