Currently in the module if the email is not configured properly meaning that they have credentials in the config for mail service but the mail service does not actually send mail. The send mail silently fails in the background. But in the UI the user sees nothing but the prompt that the telehealth can’t start right now. Nothing indicates that the email failed.
The second reason that I am writing this is that the module is not built to have or manage its own dependencies. The module depends on the dependencies from the host codebase. That means that the async needs to go into the core and not just the module. I wanted to ask the opinion of the lead project lead of what are the chances of getting this package into the core.
The package has been around for 6 years and seems to have active support. 2k+ stars. This could help with other situations where one operation can be executed first and completed before launching another.
@mdsupport I like your thoughts also. Are you already using this library?