How to update a Select property

[MerlinDE]

Whenever I try to update a PageProperty::Select on a database object, Notion throws a validation_error at me.

Reading Notion's API documentation, I understand that I have to first query a database for a page and its properties. Inside the response the possible values for the select will be listed. And I assume that I'll have to copy them to my request to set the select property.

Looking at the notion-client documentation and code, I don't see a way to do this.

Is this a missing feature or am I getting something wrong?

This is my skeleton for handling reading/writing a page in Notion:

#[derive(Debug)]
pub struct Article {
    page: Page,
}

impl Article {
    pub async fn new(database_id: &str, client: &Client) -> ArticleResult<Self> {
        let req = CreateAPageRequest {
            parent: Parent::DatabaseId {
                database_id: database_id.to_string(),
            },
            icon: None,
            cover: None,
            properties: Default::default(),
            children: None,
        };
        let page = client.pages.create_a_page(req).await?;
        Ok(Self { page })
    }
    
    pub fn type_(&self) -> Option<&String> {
        self.select_property("Type")
    }

    pub fn set_type(&mut self, type_: Option<&str>) -> ArticleResult<&mut Self> {
        self.set_select_property("Type", type_)?;
        Ok(self)
    }
    
    /// Read a select property
    fn select_property(&self, property: &str) -> Option<&String> {
        self.page.properties.get(property).and_then(|prop| {
            if let Select { id: _, select } = prop {
                select.as_ref().and_then(|s| s.name.as_ref())
            } else {
                None
            }
        })
    }

    /// Update a select property
    fn set_select_property(&mut self, property: &str, select: Option<&str>) -> ArticleResult<&mut Self> {
        let select = select.map(|s| s.to_string());

        self.page.properties.insert(property.to_string(), PageProperty::Select {
            id: None,
            select: Some(SelectPropertyValue {
                name: select,
                ..Default::default()
            }),
        });

        Ok(self)
    }
    
    /// # Updates the article in Notion
    pub async fn update(&mut self, client: &Client) -> ArticleResult<&mut Self> {
        let p: BTreeMap<String, Option<PageProperty>> = self.page.properties.clone().into_iter().filter_map(|(name, prop)| match Self::is_set(&prop) {
            (true, true) => Some((name, Some(prop))),
            (true, false) => Some((name, None)),
            _ => None,
        })
            .collect();

        if p.is_empty() {
            return Ok(self);
        }

        let mut r = UpdatePagePropertiesRequestBuilder::default();
        r.properties(p);
        match r.build() {
            Ok(req) => {
                let page = client
                    .pages
                    .update_page_properties(self.id(), req)
                    .await?;
                self.page = page;
                Ok(self)
            }
            Err(e) => {
                Err(ArticleError::SettingPropertyError {
                    property: e.to_string(),
                })
            }
        }
    }


    /// # Checks if a page property has been set (id is None) and if the property is set (value is not None)
    fn is_set(property: &PageProperty) -> (bool, bool) {
        match property {
            PageProperty::Checkbox { id, .. } => (!id.is_some(), true),
            PageProperty::Select { id, select, .. } => (!id.is_some(), select.is_some()),
            PageProperty::MultiSelect { id, multi_select, .. } => (!id.is_some(), !multi_select.is_empty()),
            PageProperty::Title { id, title, .. } => (!id.is_some(), !title.is_empty()),
            PageProperty::Relation { id, relation, .. } => (!id.is_some(), !relation.is_empty()),
            PageProperty::Date { id, date } => (!id.is_some(), date.is_some()),
            PageProperty::RichText { id, rich_text, .. } => (!id.is_some(), !rich_text.is_empty()),
            PageProperty::Url { id, url, .. } => (!id.is_some(), url.is_some()),
            PageProperty::Status { id, status, .. } => (!id.is_some(), status.is_some()),
            _ => (false, false),
        }
    }

}

[takassh]

@MerlinDE Thank you for providing code! let me work on this soon ;)