Change maxParallelSession type from Option<StringNumber> to `Opti…

…on<i32>`

CHANGELOG.md

@@ -58,6 +58,12 @@ Versioning](https://semver.org/spec/v2.0.0.html).
   set to an empty string were not saved to the `config` in the database.
 - Fixed an issue where configuration conversion failures were silently ignored,
   leading to incorrect None handling.
+- The `maxParallelSession` field in `Account` has been updated to use
+  `Option<i32>`. Previously, it was returned as an `Option<StringNumber>` type.
+  The `max_parallel_sessions` type in review-database has been updated from
+  `Option<u32>` to `Option<u8>`, the related code has been revised accordingly.
+  And since GraphQL does not support unsigned integers, the type has been
+  changed to a signed type `Option<i32>`, which can cover `Option<u8>`.
 
 ## [0.24.0] - 2024-11-19
 

Cargo.toml

@@ -25,7 +25,7 @@ num-traits = "0.2"
 reqwest = { version = "0.12", default-features = false, features = [
   "rustls-tls-native-roots",
 ] }
-review-database = { git = "https://github.com/petabi/review-database.git", rev = "f59bef0" }
+review-database = { git = "https://github.com/petabi/review-database.git", rev = "3be5e9f" }
 roxy = { git = "https://github.com/aicers/roxy.git", tag = "0.3.0" }
 rustls = { version = "0.23", default-features = false, features = [
   "ring",

src/graphql/account.rs

@@ -7,7 +7,7 @@ use anyhow::anyhow;
 use async_graphql::connection::OpaqueCursor;
 use async_graphql::{
     connection::{Connection, EmptyFields},
-    Context, Enum, InputObject, Object, Result, SimpleObject, StringNumber,
+    Context, Enum, InputObject, Object, Result, SimpleObject,
 };
 use chrono::{DateTime, NaiveDateTime, TimeZone, Utc};
 use review_database::{
@@ -173,7 +173,7 @@ impl AccountMutation {
         language: Option<String>,
         theme: Option<String>,
         allow_access_from: Option<Vec<IpAddress>>,
-        max_parallel_sessions: Option<u32>,
+        max_parallel_sessions: Option<u8>,
     ) -> Result<String> {
         let store = crate::graphql::get_store(ctx).await?;
         let table = store.account_map();
@@ -673,10 +673,8 @@ impl Account {
             .map(|ips| ips.iter().map(ToString::to_string).collect::<Vec<String>>())
     }
 
-    /// The max sessions that can be run in parallel in string within the
-    /// representable range of `u32`.
-    async fn max_parallel_sessions(&self) -> Option<StringNumber<u32>> {
-        self.inner.max_parallel_sessions.map(StringNumber)
+    async fn max_parallel_sessions(&self) -> Option<i32> {
+        self.inner.max_parallel_sessions.map(i32::from)
     }
 }
 
@@ -754,8 +752,8 @@ struct UpdateAllowAccessFrom {
 /// The old and new values of `maxParallelSessions` to update.
 #[derive(InputObject)]
 struct UpdateMaxParallelSessions {
-    old: Option<u32>,
-    new: Option<u32>,
+    old: Option<u8>,
+    new: Option<u8>,
 }
 
 struct AccountTotalCount;