Fix/470

[GrigoreMihai]

Summary

Changed the javascript rendering in favor of php side rendering for the pro charts lock.

Updated the pop-up links.

Added small changes to keep the code DRY.

Will affect visual aspect of the product

NO

Test instructions

Check that the pro charts lock notice is working as expected when a license is expired.

Check before Pull Request is ready:

• I have written a test and included it in this PR
• I have run all tests and they pass
• The code passes when running the PHP CodeSniffer
• Code meets WordPress Coding Standards for PHP, HTML, CSS and JS
• Security and Sanitization requirements have been followed
• I have assigned a reviewer or two to review this PR (if you're not sure who to assign, we can do this step for you)

Closes https://github.com/Codeinwp/visualizer-pro/issues/470.

[pirate-bot]

Plugin build for 187c7f6 is ready 🛎️!

• Download build

[preda-bogdan]

The code looks good, I did left a small change that I think we can make to improve this further and some small NIT's that you can decide if they are worth adding.

[preda-bogdan]

A small question related to this PR. Would it be too time-consuming to ensure these changes have some testing?

[preda-bogdan]

@GrigoreMihai Can you please fix the code style, the lint is failing. Thank you!

[GrigoreMihai]

@preda-bogdan I also added a test. I was also going to lint the code when pushing the test it was not ready for review when you looked.

It is all completed now, thank you for your patience!

[pirate-bot]

🎉 This PR is included in version 3.11.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀