-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update from 0.13.2.1 to 0.17.1.1 and now our data is "gone" #114
Comments
Ups wrong account i posted from, this is my issue not GameMonkey :) GameMonkey is a friend. |
@rasmusfjord Strange. I can't exactly put my finger on the problem. Off the top of my head I'm thinking...
It none of the above lead you anywhere, try attaching a debugger to this line and see |
Hey @kjac I forgot about the sql index dll so i quickly rebuilded it against the new version, but it didnt change anything, check logs, nothing here, so i did a custom build just with a logger on the "Get" method you talked about and the data is good. LogHelper.Info(JsonConvert.SerializeObject(rows)); This is the data for the form and its all there just not being displayed. here is the form, and there are no console errors in the browser, and there is no data api call on the fields. But something ive noticed changed since v.13 to v. 17 is the _sortName field is the one being used to call out the alias of the field, we have some minor stuff using the field besides the form it self so had to change it there, could it be something like that? |
@rasmusfjord might be. Or it might be something with an underscore prefix that was introduced a while back, but possibly never tested properly with the SQL index in an upgrade scenario. I'll try to reproduce it as soon as I can. |
@rasmusfjord It's the underscore issue 😞 I've got at fix for it, but it'll be a day or two before I can test it thoroughly - then I'll release a new version. |
@kjac you are the man Kenn! |
@rasmusfjord the issue is fixed in the next release, but GitHub is being childish at the moment, so I can't publish the release just now... hopefully soon. |
@rasmusfjord There. Finally. It's released 😄 thanks for the issue report 👍 |
@kjac Wow thank you Kenn for reacting so fast! |
and now it works again :) |
Awesome! |
So ive just updated this solution, and now my data entries are "gone", but think they are just not being displayed.
If i dig into the api controller it also just returns null as values, any idea why or how to fix it? This is not put into production yet still on the stage server.
We are running the SQL implementation (Yeah i know, we are not doing this on the newer solutions)
The text was updated successfully, but these errors were encountered: