-
-
Notifications
You must be signed in to change notification settings - Fork 42
Description
Currently, if you create/update/delete a post, and the content store throws an error, Indiekit doesn’t reflect this in post data, and shows a post as published; post data does not always reflect the current state of a post in the content store.
As part of ensuring the robustness of Indiekit’s data, consider the following:
Creating a post
store.createFilemethods should return the file location of the stored file, nottrue- On receipt of a file location, update a post‘s properties:
- add
storeProperties.publishedwith date if successful,falseif failed. - add
storeProperties.urlandstoreProperties.path
- add
- Show the post in the UI with the ‘Published’ state.
If an error occurs storing content, a post will have false for store.published. We already show an error message, can now show a ‘Not stored’ status, or similar.
Updating a post
store.updateFilemethods should return the updated file location of the stored file, nottrue- On receipt of a file location, update a post‘s properties:
- add
storeProperties.updatedwith date if successful,falseif failed. - update
storeProperties.urlandstoreProperties.pathif relevant
- add
If an error occurs storing content, a post will not have the properties.updated property. We already show an error message but can now show ‘Updates not stored’ status, or similar.
Deleting a post
store.deleteFilereturns a booleantrue(no change)- On receipt of
true, update a post‘s properties:- add
storeProperties.deletedwith date if successful,falseif failed. - delete
storeProperties.url
- add
If an error occurs storing content, a post will not have a store.deleted property and will show an error message. We already show an error message but can now show ‘Not deleted from store’ status, or similar.
To do
- Update store-plugins to return file location when creating/updating #661
- Update actions to add/update
storeProperties.*properties after file storage action - Update UI to reflect failure states
- Update UI to allow for retry
On these last 2, perhaps it’s less a case of providing explicit statuses, but providing a persist message on any post that has encounter issues while being saved to the content store, with an option to retry the last action?