Skip to content

Advertisment replace axios with fetch#1692

Merged
evanugarte merged 6 commits intodevfrom
advert-rem-axios
Jun 22, 2025
Merged

Advertisment replace axios with fetch#1692
evanugarte merged 6 commits intodevfrom
advert-rem-axios

Conversation

@DanielTQuach
Copy link
Copy Markdown
Contributor

#1691
replacing axios + chained promises with just fetch and await

@DanielTQuach DanielTQuach changed the title Advert rem axios Advertisment replace axios with fetch Jun 12, 2025
Copy link
Copy Markdown
Collaborator

@thebeninator thebeninator left a comment

Choose a reason for hiding this comment

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

lgtm

@evanugarte evanugarte merged commit b21a9a3 into dev Jun 22, 2025
4 checks passed
@evanugarte evanugarte deleted the advert-rem-axios branch June 22, 2025 00:42
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