feat(settings): add /settings get and update

now supports only displayName change HOW-69

src/main.rs

@@ -61,7 +61,6 @@ fn run() -> Result<(), failure::Error> {
 fn create_server(db_url: String) -> Result<(), failure::Error> {
     env_logger::init();
     use self::app_state::DbExecutor;
-    use actix_base::{SyncArbiter, System};
 
     let cpus = num_cpus::get();
     let system = System::new("htc-server");

src/models/user.rs

@@ -1,5 +1,5 @@
 use crate::schema::users;
-use crate::views::{EncodableUserPrivate, EncodableUserPublic};
+use crate::views::{EncodableUserPrivate, EncodableUserPublic, UserSettings};
 use diesel::prelude::*;
 
 #[derive(Queryable, Associations, Identifiable, Debug)]
@@ -34,6 +34,15 @@ impl User {
         }
     }
 
+    pub fn encodable_settings(self) -> UserSettings {
+        let User { display_name, .. } = self;
+
+        UserSettings {
+            display_name,
+            gravatar_email: None,
+        }
+    }
+
     pub fn find_by_id(conn: &PgConnection, user_id: i32) -> Option<Self> {
         use crate::schema::users::dsl::*;
 

src/routes/account.rs

@@ -32,29 +32,52 @@ pub fn create(
         .responder()
 }
 
-fn update(state: State<AppState>, update: Json<AccountUpdate>) -> FutureResponse<HttpResponse> {
-    #[derive(Serialize)]
-    #[serde(rename_all = "camelCase")]
-    struct R {
-        user: views::EncodableUserPrivate,
-    }
+#[derive(Deserialize)]
+#[serde(rename_all = "camelCase")]
+struct Update {
+    display_name: Option<String>,
+}
+
+#[derive(Serialize)]
+#[serde(rename_all = "camelCase")]
+struct SettingsResponse {
+    settings: views::UserSettings,
+}
 
+fn update(
+    state: State<AppState>,
+    auth: Auth,
+    update: Json<Update>,
+) -> FutureResponse<HttpResponse> {
     state
         .pg
-        .send(update.0)
+        .send(AccountUpdate {
+            requester_id: auth.user.id,
+            display_name: update.display_name.clone(),
+        })
         .from_err()
         .and_then(|res| match res {
             Ok(user) => Ok(answer_success!(
                 Ok,
-                R {
-                    user: user.encodable_private()
+                SettingsResponse {
+                    settings: user.encodable_settings()
                 }
             )),
             Err(err) => Ok(err.error_response()),
         })
         .responder()
 }
 
+pub fn settings(auth: Auth) -> FutureResponse<HttpResponse> {
+    futures::future::ok(answer_success!(
+        Ok,
+        SettingsResponse {
+            settings: auth.user.encodable_settings()
+        }
+    ))
+    .responder()
+}
+
 /// POST /account/session
 pub fn login(
     login_data: Json<SessionCreate>,
@@ -105,7 +128,10 @@ pub fn scope(scope: Scope<AppState>) -> Scope<AppState> {
     scope
         .resource("/", |r| {
             r.post().with(self::create);
-            r.get().with(self::update);
+        })
+        .resource("/settings/", |r| {
+            r.get().with(self::settings);
+            r.put().with(self::update);
         })
         .resource("/session/", |r| {
             r.post().with(self::login);

src/views.rs

@@ -16,3 +16,11 @@ pub struct EncodableUserPublic {
     pub display_name: Option<String>,
     pub id: i32,
 }
+
+/// User settings to communicate with frontend
+#[derive(Debug, Serialize, Deserialize)]
+#[serde(rename_all = "camelCase")]
+pub struct UserSettings {
+    pub display_name: Option<String>,
+    pub gravatar_email: Option<String>,
+}