Hendrik Luehrsen

Hey @eliazara73,

I am not sure if I understand what you’re trying to build? Do you maybe have an example?

Thanks! ??

Hey @meeraprajapati05,

thank you for your suggestion!

We are currently changing the way the input works and basically make it a free text field. That would make things easier.

Expect a patch later today!

Best,
Hendrik

Hey @colecoboy84,

you are correct, the input was built with european/german addresses in mind. But your fix should be okay.

You can always check the validity of the event bock with the structured data test tool by google:

https://search.google.com/structured-data/testing-tool/u/0/#url=https%3A%2F%2Fblocks.wp-munich.com%2Fblocks%2Fevent%2F

Shooting from the hip here, but this should pretty much do the trick:

https://gist.github.com/Luehrsen/abd60d92d65cf71cbc513323171138dc

Can you maybe send me a link to such a site? Maybe I can quickly write a drop in function for you to put into the functions.php.

• This reply was modified 6 years ago by Hendrik Luehrsen.

Amazing! With your feedback we have just released 0.5.0!

We would love to hear your feedback if it works any better.

Thank you! You can find the additional code for WooCommerce here:
https://github.com/luehrsenheinrich/wp-quicklink/blob/master/build/plugin-compatibility/woocommerce.php

• This reply was modified 6 years ago by Hendrik Luehrsen.

We have started working on WC compatibility.

https://github.com/luehrsenheinrich/wp-quicklink/pull/16/files

Would you guys be able to beta-test this, if we provide a beta-version as a zip file?

@camilo517 It is precisely the issue. But it needs some refactoring to make it work properly. ??

For reference: https://github.com/luehrsenheinrich/wp-quicklink/pull/14

I can confirm.

I will try to create a patch within the next week.

Hey @fka,

thanks for the suggestion. We will definitely take this into account when pushing the next features, as we were also noticing this flaw ourselves.

Best,
Hendrik

Hm. This one is so obvious, I can’t figure out why we did not think of it.

Well add it in the next patch, give us a few days.

Thanks for your input! ??

This is definitely a great idea. A GUI to modify some options is planned, for now you can use a filter to add mission critical URLs to the ignore list.

Thank you for your feedback!

Hey,

any client based tracking should not be fired, as Quicklink only preloads the html and does not interpret or run code. (Like JavaScript.)

You will see some more hits on your Server logs though.

The switch is called ?fixed background“ in the layout panel of the background block. ??

Hey Guido,

i‘ve written the fix yesterday. Patch coming soon!

Thanks!

Hendrik