Skip to content
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

fix: media manager, error response schema parsings and other minor fixes #28

Merged
merged 20 commits into from
May 29, 2024

Conversation

sarthakjdev
Copy link
Owner

@sarthakjdev sarthakjdev commented May 16, 2024

Issue #26

  • We have removed support for phone number manager as of now, the reason is that this library is intended to be a single client library linked linked with a single phone number. So if a client is already initiated with a phone number, this would be fundamentally wrong to use to verify its sibling, instead this would have been a parent client without phone number dependency -> verifying phone number and selection. We can do that, the code is written dynamically. But for the sake of simplicity, we will keep it this way, only and if somebody wants to add a phone number they still have to use the meta business platform, for which guides will be added soon to /guide of website.
  • Support added for getUrl and delete media.
  • Proper error handling and response schema for the send and other functions used.
  • Fixes ESM support bug

…ager methods

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
…gers

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
@sarthakjdev sarthakjdev self-assigned this May 27, 2024
Copy link

🚨 Linter failed

Copy link

🟢 Deployment success:

Deployed Url: https://wapijs-4pxa9kcwb-sarthak-jains-projects-940d0d6f.vercel.app

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

@sarthakjdev sarthakjdev changed the title fix: return typings generics and managers implementation fix: managers implementation May 27, 2024
@sarthakjdev sarthakjdev changed the title fix: managers implementation fix: managers May 27, 2024
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

… complexity to the lib as of now, will add if somebody needs it.

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

@sarthakjdev sarthakjdev changed the title fix: managers fix: media manager, error response schema parsings and esm support bug May 29, 2024
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Copy link

🚨 Linter failed

Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
Signed-off-by: sarthakjdev <jsarthak448@gmail.com>
@sarthakjdev sarthakjdev marked this pull request as ready for review May 29, 2024 18:16
Copy link
Owner Author

@sarthakjdev sarthakjdev left a comment

Choose a reason for hiding this comment

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

LOOKS OK!

@sarthakjdev sarthakjdev changed the title fix: media manager, error response schema parsings and esm support bug fix: media manager, error response schema parsings and other minor fixes May 29, 2024
@sarthakjdev sarthakjdev merged commit 6feec43 into master May 29, 2024
5 checks passed
github-actions bot pushed a commit that referenced this pull request May 29, 2024
# [create-wapi-app-v0.1.1](create-wapi-app-v0.1.0...create-wapi-app-v0.1.1) (2024-05-29)

### Bug Fixes

* media manager, error response schema parsings and other minor fixes  ([#28](#28)) ([6feec43](6feec43))
@sarthakjdev
Copy link
Owner Author

🎉 This PR is included in version create-wapi-app-v0.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request May 29, 2024
# [@wapijs/wapi.js-v0.0.8](https://github.com/sarthakjdev/wapi.js/compare/@wapijs/wapi.js-v0.0.7...@wapijs/wapi.js-v0.0.8) (2024-05-29)

### Bug Fixes

* media manager, error response schema parsings and other minor fixes  ([#28](#28)) ([6feec43](6feec43))
@sarthakjdev
Copy link
Owner Author

🎉 This PR is included in version @wapijs/wapi.js-v0.0.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant