Skip to content

API_HANDLER Module updated for only handling API #438

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

Merged
merged 1 commit into from
Jun 6, 2025

Conversation

sue-lim
Copy link
Contributor

@sue-lim sue-lim commented Jun 3, 2025

Implement API handler for external exchange service into Currency Script

[FEATURE] Currency Script | Implement API handler for external exchange service #431

Summary

Implemented api_handler module to handle the API Call.

Description

The changes are as follows:

  • Updated api_handler for handling the api call and logic

Checks

in the repository

  • Made no changes that degrades the functioning of the repository
  • Gave each commit a better title (unlike updated README.md)

in the PR

  • Followed the format of the pull_request_template
  • Made the Pull Request in a small level (for the creator's wellfare)
  • Tested the changes you made

image

Thank You,
Sue

@sue-lim
Copy link
Contributor Author

sue-lim commented Jun 3, 2025

@DhanushNehru - please kindly approve PR.

@DhanushNehru DhanushNehru merged commit 62fe11a into DhanushNehru:master Jun 6, 2025
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