Open jaoaustero opened 1 month ago
The changes involve a comprehensive renaming of methods in the TawkMessenger
class within the src/lib/index.js
file, as well as updates in related documentation. All methods previously exposed through this.app.provide
now have the prefix "tawk" to standardize the naming convention across action, getter, listener, and setter methods. Similar updates were made in the API reference and setup documentation, ensuring a consistent interface and naming scheme throughout the codebase.
File | Change Summary |
---|---|
src/lib/index.js |
Renamed methods in TawkMessenger class to prefix with "tawk" for consistency: action, getter, listener, and setter methods. |
docs/api-reference.md |
Updated API callback functions and variables to prefix with "tawk" for consistency across the document. |
docs/spa-setup.md |
Changed injected function from onLoad to tawkOnLoad in the Vue.js Options API. |
docs/ssr-setup.md |
Updated injected functions from onLoad and toggle to tawkOnLoad and tawkToggle in the Vue.js Options API. |
In the code where rabbits play,
Methods danced and found their way.
With "tawk" they hop, a joyful tune,
Naming bright as a sunny moon.
Consistency now leads the cheer,
Hopping forward, no more fear! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary
There are users reported collision of provider keys since we use generic naming on our API. I added prefix
tawk
in all actions, getters, listeners, and setters. This will resolve the issue #6Summary by CodeRabbit
These updates ensure a more intuitive experience when interacting with the TawkMessenger functionalities.