We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Much like Article in #415, the Feed model mixes server-set and client-set fields. Unlike that issue the primary concern here is lost updates.
url
next_check
last_checked
last_changed
error
etag
last_modified
digest
feed_title
site_url
created
deleted
user_title
all_count
unread_count
fave_count
Split into a few models:
Feed
Defines a feed to check. Contains the client-set fields.
The url field contains the URL originally entered by the user.
FeedState
Contains the server-set fields.
The current_url field contains the URL after persisting any permanent redirects.
current_url
FeedCounts
Contains all_count, unread_count, and fave_count. Maintained exclusively by triggers?
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Much like Article in #415, the Feed model mixes server-set and client-set fields. Unlike that issue the primary concern here is lost updates.
Server-set fields
url
— may be updatednext_check
,last_checked
,last_changed
— timestampserror
etag
,last_modified
,digest
feed_title
,site_url
— feed metadataClient-set fields
url
— entered by the usercreated
,deleted
user_title
Server- and client-set fields
all_count
unread_count
fave_count
Proposed changes
Split into a few models:
Feed
Defines a feed to check. Contains the client-set fields.
The
url
field contains the URL originally entered by the user.FeedState
Contains the server-set fields.
The
current_url
field contains the URL after persisting any permanent redirects.FeedCounts
Contains
all_count
,unread_count
, andfave_count
. Maintained exclusively by triggers?The text was updated successfully, but these errors were encountered: