Async JS Improvement

  • Because of the timeouts of the feedback company api yesterday, all JS on my site was delayed. The implementation of the script for the api doesn’t seem optimal.

    To make my site work, I changed 1 line of code in your plugin yesterday.
    I can’t do a PR on the plugin, since I can’t seem to find the repo.

    Is it an to add the async tag to line 314? Like so:

    $out .= '<script type="text/javascript" src="https://www.feedbackcompany.com/widgets/feedback-company-widget.min.js" async></script>';
Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Middelkoop.cc

    (@janmiddelkoop)

    Hi @stevejonk. You’re right that this could be a nice improvement. I’ll check with Feedback Company API engineers if there’s any downsides to this we might overlook and do internal testing. If there are no problems, I’ll push a new plugin with this change.

    Thread Starter stevejonk

    (@stevejonk)

    @Middelkoop.cc, nice! Thanks for your reply!

    • This reply was modified 10 months, 1 week ago by stevejonk.
Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘Async JS Improvement’ is closed to new replies.