Meta Box

Drag and drop order reset without consent

Support › MB Relationships › Drag and drop order reset without consentResolved

Viewing 7 posts - 16 through 22 (of 22 total)
1 2
  • Author
    Posts
  • December 15, 2018 at 12:29 AM #12680
    DougDoug
    Participant

    Thanks for the video, Anh.

    I created a fresh install of WP just for this, installed Meta Box, MB Custom Post Type, MB Relationships, and the Twenty Seventeen theme, and duplicated the steps you took in your first video. And yes, it's working for me right now -- order is preserved after updating a speaker post.

    But our staging and production servers are still bumping the last updated post to the bottom of the list. I'm going to slowly add back in pieces of our staging environment (plugins, theme files) to see if I can pinpoint what it is that causes the posts to reorder. If I find what it is, I’ll report back here.

    December 15, 2018 at 12:31 AM #12681
    Content PilotContent Pilot
    Participant

    Ahn,

    These videos look great. Looks like you have the defaults ort order in the admin set to use the new database columns? I like this being the default. Once it is set, they should not change.

    I can't seem to find the github repo for the dev branch. All I see is master and check_relationship.

    December 15, 2018 at 12:33 AM #12682
    Content PilotContent Pilot
    Participant

    Doug, where are you finding the new package?

    December 15, 2018 at 12:45 AM #12684
    DougDoug
    Participant

    Update: Actually, scratch my report above. I think it's working on our staging server now too.

    Just went back to test one more time, and related post order is preserved on our staging server too. Not sure why I saw it move yesterday, since I haven’t changed any of the plugins. But this issue seems to be fixed from what I can see. Thank you!

    @Clayton - I don't think there is a separate dev branch on GitHub. Since I don't see any dev branches either, I'm betting GitHub is dev for the MB team. I just grabbed master of MB Relationships and installed from that.

    December 15, 2018 at 12:58 AM #12685
    Content PilotContent Pilot
    Participant

    Thanks Doug. I did the same.

    Ahn, ran similar tests and it looks great. Please push the updates.

    Thanks for everything! This is a huge lifesaver for my clients.

    December 15, 2018 at 1:14 AM #12686
    DougDoug
    Participant

    Ok, one more update from me. I just updated MB Relationships on our production server too. Tested by updating one of our speakers, and then checking the event, and that speaker was bumped to the bottom of the list.

    However, after dragging the speaker back to the correct position on the event page, updating, then updating the same speaker again, the order was preserved. And it's preserved every time now for that event.

    I tested our other events, here's what I found (that mirrors what Clayton notes above)...

    Any new relationships that get created with the updated plugin code should preserve order, because of the way index order is getting set now. But for any existing posts that had relationships defined before updating the plugin, you must go into each of those posts, and update each post once to establish an index order for its related posts. Then specified post order will be preserved from that point forward.

    December 19, 2018 at 5:46 PM #12738
    Anh TranAnh Tran
    Keymaster

    Hey guys,

    Thanks a lot for your tests! And sorry for the delay reply.

    You're right about the order of existing posts. Since we upgrade the database structure, all existing posts won't have the order. So you need to edit them once to trigger the order change. Then everything else will work as expected.

    I'll update the plugin now 🙂

    PS: I use the master branch for developing. No dev branch at the moment.

  • Author
    Posts
Viewing 7 posts - 16 through 22 (of 22 total)
1 2
  • You must be logged in to reply to this topic.