Merge pull request #3 from rahulkp220/feat/enum_for_sorting

Use enum for sort

examples/run.rs

@@ -1,4 +1,4 @@
-use ordiscan::{GetInscriptionInfoParams, GetListOfInscriptionParams, Ordiscan};
+use ordiscan::{GetInscriptionInfoParams, GetListOfInscriptionParams, Ordiscan, Sort};
 
 #[tokio::main]
 async fn main() -> Result<(), Box<dyn std::error::Error>> {
@@ -22,8 +22,8 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
   let list_of_inscriptions = ordiclient
     .get_list_of_inscriptions(GetListOfInscriptionParams {
       address: Some("bc1qxy2kgdygjrsqtzq2n0yrf2493p83kkfjhx0wlh"),
+      sort: Sort::InscriptionNumberDesc,
       content_type: None,
-      sort: None,
       after_number: None,
       before_number: None,
     })

src/lib.rs

@@ -23,7 +23,7 @@ pub struct Ordiscan {
 pub struct GetListOfInscriptionParams<'a> {
   pub address: Option<&'a str>,
   pub content_type: Option<&'a str>,
-  pub sort: Option<&'a str>,
+  pub sort: Sort,
   pub after_number: Option<usize>,
   pub before_number: Option<usize>,
 }
@@ -34,6 +34,21 @@ pub struct GetInscriptionInfoParams<'a> {
   pub number: Option<usize>,
 }
 
+#[derive(Debug)]
+pub enum Sort {
+  InscriptionNumberDesc,
+  InscriptionNumberAsc,
+}
+
+impl Sort {
+  pub fn as_str(&self) -> &'static str {
+    match self {
+      Sort::InscriptionNumberDesc => "inscription_number_desc", // default
+      Sort::InscriptionNumberAsc => "inscription_number_asc",
+    }
+  }
+}
+
 impl<'a> Ordiscan {
   // create a new Ordiscan client
   pub fn new(key: String) -> reqwest::Result<Self> {
@@ -86,9 +101,15 @@ impl<'a> Ordiscan {
     params: GetListOfInscriptionParams<'a>,
   ) -> Result<Vec<schema::InscriptionInfo>> {
     let header = format!("Bearer {}", self.api_key);
-    let sort = params.sort.unwrap_or("inscription_number_desc");
-    let mut url =
-      Url::parse(format!("{}/inscriptions?sort={}", API_BASE_URL, sort).as_str()).unwrap();
+    let mut url = Url::parse(
+      format!(
+        "{}/inscriptions?sort={}",
+        API_BASE_URL,
+        params.sort.as_str()
+      )
+      .as_str(),
+    )
+    .unwrap();
 
     // TODO make this look better
     // dynamically create query params