Plugins Intro Tour Tutorial Support Tour of new post breaks ...
Notifications
Clear all

[Solved] Tour of New Post Breaks

34 Posts
2 Users
1 Likes
1,677 Views
Travis
Posts: 25
Customer
Topic starter
(@travis-wooley)
Eminent Member
Joined: 1 year ago

Sorry but no change at all. Here is a screen recording from my live site.

Reply
1 Reply
Travis
Customer
(@travis-wooley)
Joined: 1 year ago

Eminent Member
Posts: 25

Hmm, it seems it is not quite so simple. The above does work now. I must have gotten caught by some kind of cache. However, longer runs that update more than just the title still error. I'll test a longer run again in a few hours to ensure that wasn't cache too.

Reply
Travis
Posts: 25
Customer
Topic starter
(@travis-wooley)
Eminent Member
Joined: 1 year ago

Tested again longer runs still don't work. 🙁

Also, I checked and confirmed that it is not the clicking Publish that causes the problem. Something before then is causing the problem. Here is a screen recording of one of your demo sites showing the error even though the tour ends before I click publish.

Reply
Travis
Posts: 25
Customer
Topic starter
(@travis-wooley)
Eminent Member
Joined: 1 year ago

Good news! I found two triggers, and I suspect the source is the same for both triggers. Turning on "VARIABLES IN URL" causes this issue as does "Lock by URL Parameter". I suspect there is a bug in the URL parsing code that causes the URLs on the admin side to get mangled. I was able to reliably make the issue occur and stop occurring by toggling the "VARIABLES IN URL" option.

 

Reply
4 Replies
Deep Presentation
(@admin)
Joined: 3 years ago

Member
Posts: 90

@travis-wooley,

Perfect, thank you very much for your testing. This really helps as now I have an idea where to find a bug. I'll be back as soon as I'm able to fix it.

Reply
Travis
Customer
(@travis-wooley)
Joined: 1 year ago

Eminent Member
Posts: 25

@admin Just checking in. Any progress yet?

Reply
Deep Presentation
(@admin)
Joined: 3 years ago

Member
Posts: 90

Hey @travis-wooley,

Sorry for late reaction. We were migrating our web to faster server so I've been very busy.

Sadly I'm not able to reproduce it at my local. Would you mind to temporarily share an admin access so I can check it at your web? If so, please send me credentials at support@deeppresentation.com

Did you try deactivate all unnecessary plugins to test, if it is not a case of plugin unwanted interaction? 

Best regards

Tomas

Reply
Travis
Customer
(@travis-wooley)
Joined: 1 year ago

Eminent Member
Posts: 25

@admin Done.

I gave you access to our test enviornment so you can modify things as much as you would like without concern for our live site. (It is a very recent clone of our live site.)

When you log in you will see a series of icons. Virtually all of those icons lead to tours. The "Weekly Service (Pre-Service Setup)" is the most consistently broken one. I have never been able to get that one to work correctly. It has always exhibited the save error. I have also received reports that the very simple "Update Bulletin" one has the same problem. The file upload works fine but it won't save the page afterwards. The others may or may not work I haven't confirmed in the past few weeks.

Let me know if you have any questions or if you get it all fixed. 🙂

Reply
Deep Presentation
Posts: 90
(@admin)
Member
Joined: 3 years ago

So I found an issue. The problem is that WordPress admin has problem when query strings starts with "/?" ... it requires only "?". However in general - regarding this right way of query string is "/?". That is why our plugin force it, but WordPress has obviously problem with it.

https://watch.screencastify.com/v/k2vXAJh4iJYba52a4XyB

So I identified it and now I'll make a fix.

I'll keep you updated and inform you when release is up.

I see that there would be nice to add feature ... go to next after click on visualized element ... so user doesn't have to click next and all tour would be smoother. I'll think about that and add it to todo.

Btw I really like how you use our plugin and these are beautiful use cases of admin integration. Would you mind me to use some screen video of tours on your site for our marketing purposes? I can hide whatever you wouldn't like to publish ... branding etc.

 

 

Reply
2 Replies
Travis
Customer
(@travis-wooley)
Joined: 1 year ago

Eminent Member
Posts: 25

@admin Great! Thanks so much!

I'm afraid I couldn't see what you were trying to show in the screencast I think it cut off the top part. However, I'm very excited that you were able to identify the cause! Also, adding an "auto next" feature like that would be great!

I'm glad you like the tours! They have been very helpful for me. Sure you can use some screen videos.

Reply
Deep Presentation
(@admin)
Joined: 3 years ago

Member
Posts: 90

@travis-wooley,

Thank you for permission to take some screen videos. First I'll implement "auto next" feature so tours are more intuitive, then I'll take some video screenshots. I'll keep you updated, when "auto next" feature is implemented and also everything is properly tested and new version released.

Reply
Deep Presentation
Posts: 90
(@admin)
Member
Joined: 3 years ago

It is fixed. It is still just beta version, that I uploaded to https://tourtesting.wpengine.com ... I need to properly test it before I release it officially as it was quite a change to plugin's logic. 

Let me know if all your tours are fixed, I tested just few of them.

Reply
2 Replies
Travis
Customer
(@travis-wooley)
Joined: 1 year ago

Eminent Member
Posts: 25

@admin Thank you so much!

I just did a quick run through of a couple and the only thing I noticed is when I click publish the tour always ends, previously it would continue. Not a big deal for me I can just rearrange so that publish is always the last step.

Looking forward to "auto next" too. 🙂

Reply
Deep Presentation
(@admin)
Joined: 3 years ago

Member
Posts: 90

Hi @travis-wooley,

  • I see, as now publish works (was not before due bug), it will redirect from post-new.php to post.php and tour doesn't know it. However I'm thinking about solution of these and it will probably come in next releases.
  •  "auto next" feature is released. You can set it up under interaction options for each step.

Have a nice day

Tomas

Reply
Page 3 / 4
Share:
Top chevron-right-circle linkedin facebook pinterest youtube rss twitter instagram facebook-blank rss-blank linkedin-blank pinterest youtube twitter instagram