-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
All the "Featured Stores" as a category in the Order List moved to the "Exclude" side seemingly on their own, except 2 of them #208
Comments
im not sure what im looking for, it looks ok ?
Chad Gladue
*DevNull Technology, LLC*
…On Sun, Jun 11, 2017 at 11:22 AM, robgpeak ***@***.***> wrote:
@cgladue <https://github.com/cgladue> before I go in and fix it, I wanted
to see if you might know why this would have happened? Look at the home
page, and then https://shop.complinks.co/admindashboard/advertisers/
setAdvertiserDisplayOrder/153
It seems to be the only category this happened with.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#208>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AHjClQtz4bkBN7-A3nY58b2-3vU73gSdks5sDAZRgaJpZM4N2bNq>
.
|
oh, remember you said if the display order is 0 then dont show them, its
because all the other ones are in the excluded column (that means display
order = 0)
Chad Gladue
*DevNull Technology, LLC*
…On Sun, Jun 11, 2017 at 11:51 AM, robgpeak ***@***.***> wrote:
There are only two in the "Included" section, but there were 18 or so when
Morgan finished on Friday. I am not sure what happened...
[image: screen shot 2017-06-11 at 11 49 49 am]
<https://user-images.githubusercontent.com/28393235/27012362-46dc4f46-4e9c-11e7-91c5-2fce8659bf41.png>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#208 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHjClSvrQIB1csAkT9BYo7KaYjHfPJEtks5sDAz4gaJpZM4N2bNq>
.
|
Yeah, but they WERE dragged into the "Included" side, and ordered in the right order, and they were showing up fine on the front end at the end of the day on Friday when her and I looked at it, and then something seems to have happened, and now only those two are showing on the Included side, and now only those two are showing on the front end. |
@cgladue Moran just told me that the other potential issue was that she added new header images to all of those that were in "included" and then they subsequently move to excluded on their own after she added a new header image. Best Buy and Autozone header images were added a few days earlier, and that is the reason they were not moved over to excluded. So that seems to be the issue some how... |
@cgladue So it is also happening site wide. Meaning, just now, @morganpeak added a new header image for a specific store "Zales" which prior to her adding the header image to that specific store page, WAS in the "Included" (left) side of the Order List, then after she added the header image, it moved itself over to the "Excluded" (right) side. |
ok, i am looking at it right now, i am probbally overwriting that field with a 0 when saving .. hold off the header images for a few till i look at this more... |
@robgpeak @morganpeak i believe i found the bug, i was removing all categories and re-adding them each time an advertiser was saved, therefore resetting to default of 0 when saving .. now i only remove if not in the category anymore. |
@cgladue before I go in and fix it, I wanted to see if you might know why this would have happened? Look at the home page, and then https://shop.complinks.co/admindashboard/advertisers/setAdvertiserDisplayOrder/153
It seems to be the only category this happened with.
The text was updated successfully, but these errors were encountered: