Skip to content
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

"additionalnfo" is missing from all originalStory Data (from the excel) #235

Open
rotem12345678 opened this issue Nov 8, 2020 · 4 comments
Assignees
Labels

Comments

@rotem12345678
Copy link
Collaborator

rotem12345678 commented Nov 8, 2020

when doing
http://localhost:5000/api/getStoryForEdit?originalStory=5f96df576c491f0f3064bfc3

the field "additionalnfo" is missing in originalStory but existed in ModeratedStory

This happens for all original stories that were converted from the original excel file (all but id: 5f9dc71d4e1e4100375c9e19 that was inserted afterwards).

This should be fixed by copying this field from the moderated stories db into the public stories db (no api for that).
While you're at it, please also fix the createdDate field into 15/5/2020
and delete 3 fake/tests testimonies that were inserted on 19/11

@orlylev
Copy link
Collaborator

orlylev commented Nov 16, 2020

This specific story does not have the files in the DB
In general, stories have it. What do you expect in this case?

@rotem12345678
Copy link
Collaborator Author

we expect that If the field existed in ModeratedStory it should exist in OriginalStory too.

@orlylev
Copy link
Collaborator

orlylev commented Nov 16, 2020

I checked again looks like the additionalnfo info is an empty string and it returns as an empty string by the BE API

@orlylev orlylev removed their assignment Nov 16, 2020
@moranw
Copy link
Collaborator

moranw commented Nov 16, 2020

@orlylev I checked the original excel and there is data there and it is very important data, maybe it was some corruption on the data transfer method?
I see that the excel file is filled with this data, but no original story has it (as seen on the admin side)!
The hebrew title for this field was: בכמה משפטים, מה הכי היית רוצה לומר למי שבמצב דומה?
I'm sharing with you the original excel again

@moranw moranw changed the title "additionalnfo" is missing in originalStory Data "additionalnfo" is missing from all originalStory Data (from the excel) Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants