Allow partial person details (#651)

* fix(frontend): add correct check for default value

* refactor(backend): change order of functions

* refactor(backend): remove indirection of person details

* feat(frontend): re-order only if old order is not new order

* feat(frontend): close re-order drawer only if there is a change

* feat(backend): get name in partial person model

* feat(backend): get person name from anilist

* feat(backend): get person name from audible

* feat(backend): get person name from igdb

* feat(backend): get person name from manga updates

* feat(backend): get person name from vndb

* feat(database): add `is_partial` field to person table

* fix(backend): do not hard unwrap person name

* fix(backend): create seen items correctly

* fix(frontend): show input on correct place

* ci: remove healthcheck from dockerfile

* fix(database): set all people as not partial

* feat(backend): do not update person every time

* feat(backend): make getting details for person

* feat(backend): update person if it is partial

* refactor(backend): change name of trait members

* feat(backend): return `is_partial` for media and person items

* feat(frontend): display text if media is partially downloaded

* feat(frontend): redirect to correct page after marking as watched

* fix(frontend): show `Show Details` btn on podcasts too

* feat(backend): mutation to update person details

* feat(frontend): add action to update person

* fix(backend): update `is_partial` value when updating people

* fix(backend): extend possible jobs for people

* fix(backend): associate media with people

* build(backend): bump version

* feat(database): add `created_on` field to `user_to_entity`

Also update it with the correct values for exercises.

* feat(database): update media items correctly

* fix(database): take user_id into account when updating column

* fix(database): set created_on to the least of seen or added to collection

* fix(database): also take the last updated on into account

* feat(backend): return `created_on` for user exercise details

* feat(frontend): display when the exercise was first done on

* chore(database): change name of migration

* fix(database): change migration for updating created_on

* feat(database): change varchar columns to text

* fix(database): change all rust enum columns to text

* style(database): add indentation

* ci: update rust toolchain

Author: IgnisDa

Cargo.lock

@@ -56,8 +56,6 @@ COPY --from=frontend-builder --chown=ryot:ryot /app/apps/frontend/node_modules .
 COPY --from=frontend-builder --chown=ryot:ryot /app/apps/frontend/package.json ./package.json
 COPY --from=frontend-builder --chown=ryot:ryot /app/apps/frontend/build ./build
 COPY --from=backend-builder --chown=ryot:ryot /app/ryot /usr/local/bin/ryot
-HEALTHCHECK --interval=5m --timeout=3s \
-  CMD curl -f http://localhost:5000/config || exit 1
 CMD [ \
     "concurrently", "--names", "frontend,backend,proxy", "--kill-others", \
     "PORT=3000 npx remix-serve ./build/server/index.js", \

Dockerfile

@@ -1,6 +1,6 @@
 [package]
 name = "ryot"
-version = "4.2.7"
+version = "4.2.8"
 edition = "2021"
 repository = "https://github.com/IgnisDa/ryot"
 license = "GPL-3.0"

apps/backend/Cargo.toml

@@ -8,14 +8,14 @@ use serde::{Deserialize, Serialize};
 use strum::Display;
 
 use crate::{
-    entities::metadata,
+    entities::{metadata, person},
     exporter::ExporterService,
     fitness::resolver::ExerciseService,
     importer::{DeployImportJobInput, ImporterService},
     miscellaneous::resolver::MiscellaneousService,
     models::{
         fitness::Exercise,
-        media::{PartialMetadataPerson, ProgressUpdateInput, ReviewPostedEvent},
+        media::{ProgressUpdateInput, ReviewPostedEvent},
         ExportItem,
     },
 };
@@ -145,8 +145,8 @@ pub enum ApplicationJob {
     ReEvaluateUserWorkouts(i32),
     UpdateMetadata(metadata::Model),
     UpdateExerciseJob(Exercise),
+    UpdatePerson(person::Model),
     RecalculateCalendarEvents,
-    AssociatePersonWithMetadata(i32, PartialMetadataPerson, usize),
     AssociateGroupWithMetadata(MetadataLot, MetadataSource, String),
     ReviewPosted(ReviewPostedEvent),
     PerformExport(i32, Vec<ExportItem>),
@@ -205,16 +205,13 @@ pub async fn perform_application_job(
             }
             true
         }
+        ApplicationJob::UpdatePerson(person) => misc_service.update_person(person.id).await.is_ok(),
         ApplicationJob::UpdateExerciseJob(exercise) => {
             exercise_service.update_exercise(exercise).await.is_ok()
         }
         ApplicationJob::RecalculateCalendarEvents => {
             misc_service.recalculate_calendar_events().await.is_ok()
         }
-        ApplicationJob::AssociatePersonWithMetadata(metadata_id, person, index) => misc_service
-            .associate_person_with_metadata(metadata_id, person, index)
-            .await
-            .is_ok(),
         ApplicationJob::AssociateGroupWithMetadata(lot, source, group_identifier) => misc_service
             .associate_group_with_metadata(lot, source, group_identifier)
             .await

apps/backend/src/background.rs

@@ -19,6 +19,7 @@ pub struct Model {
     pub created_on: DateTimeUtc,
     pub last_updated_on: DateTimeUtc,
     pub name: String,
+    pub is_partial: Option<bool>,
     #[sea_orm(column_type = "Json")]
     #[graphql(skip)]
     pub images: Option<Vec<MetadataImage>>,

apps/backend/src/entities/person.rs

@@ -15,6 +15,7 @@ use crate::models::{
 pub struct Model {
     #[sea_orm(primary_key)]
     pub id: i32,
+    pub created_on: DateTimeUtc,
     pub last_updated_on: DateTimeUtc,
     pub user_id: i32,
     pub metadata_id: Option<i32>,