Option to export/import watch history to db. #2118
Conversation
| <label v-for="field in fields" :key="field" class="flex gap-2 items-center"> | ||
| <input | ||
| class="checkbox" | ||
| type="checkbox" | ||
| :value="field" | ||
| v-model="selectedFields" | ||
| :disabled="field === 'videoId'" | ||
| /> | ||
| <span v-text="formatField(field)" /> | ||
| </label> |
There was a problem hiding this comment.
We likely don't want this to be user selectable, as we want to stick to a standard format between frontend implementations.
We would potentially align to a new format in the distant future to allow inter-compatibility with other frontends once a format is standardized - UniversalPipeWrench/unified-user-data-format#1.
There was a problem hiding this comment.
I see one problem with this: What if the format we decide on is not what users want?
For example, if we include watchedAt and currentTime by default this might be a privacy concern for some users. On the other hand, if we decide to leave them out we are throwing out features which I would assume most users want, myself included.
The only solution I can think of is for the standard format to allow these to be optional.
There was a problem hiding this comment.
I feel currentTime is always necessary since otherwise, we can't show any indicators on videos (just the watch history page).
Regarding watchedAt, that's a fair point, but we don't know how to sort videos then.
However, I feel this is less of an issue since watch history is stored entirely locally - so, it's unlikely any third party gets access to it in the first place.
|
What's the status on this? |
|
I'd like to bump this too. Unlike the "mark as watched" button PR, this seems to be fully ready for merge. |
Closes #2117
Closes #1514