-
-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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 the data model & fixtures #101
Comments
@tgriesser I've done most of this, but I got a bit stuck trying to set published_at and published_by. I intended to use the existing on saving method on the post model, but I can't find any way to get the old values. There is a this._previousAttributes, but this contains the same as attributes. Also, I'm thinking having both created_by and author_id might be over the top... if we make it possible to change the author, we could just change the created by... and we can just name the relationship from post to user "author" rather than user. Am really interested to hear thoughts an opinions on these changes. |
We definitely will allow changing of assigned authors later - it also may be worth thinking about how post revisions (potentially by multiple authors) might work |
This is the reason why I wanted to add author_id, so that created_by refers only to who created the post, not who shows as the author. This doesn't make much sense right now, but when we have multiple versions of a post & advanced workflows, created_by will be the user who made the change, and author_id will still be who we show as the author on a post. My only concern right now was whether this would cause confusion, but I think I have just convinced myself that it is necessary. |
@ErisDS - For the About the |
Hmm.. many to many authors.. interesting idea. I'm not done with this issue yet, still have the custom data bits to do. Gonna create a separate issues for published at/by & generating uuids |
@tgriesser I've closed this, but whilst I was messing around with the tests I noticed some oddness with dates. Please see api_post_spec.js line 86&86.. I'm not sure why these assertions were finding timestamps rather than date objects? |
Further changes to the data model to ensure created_by, author_id and updated_by are all set to user 1 Updated settings such that the default type is always 'general', and changed the types in the fixtures to be slightly more useful Added additional assertions to tests to cover more assumptions about data
The post data model is currently very rough. Discussion on the topic has died, so I'm gonna put this issue out there and try to make some progress.
Posts:
A decision needs to be made about what to do with image. That will go into a separate story.
Users:
Settings:
This issue should encompass updating the two 001.js files so that we have a better model & fixtures to go with it.
The text was updated successfully, but these errors were encountered: