🐛 fix(functions.rs): fix bug in cancelOffer function where offer status was not being updated correctly

✨ feat(functions.rs): add cancellation functionality to cancelOffer function
🐛 fix(types.rs): add CANCELLED status to OfferStatus enum

Author: tlacloc

pallets/afloat/src/functions.rs

@@ -864,6 +864,19 @@ impl<T: Config> Pallet<T> {
     ensure!(<AfloatOffers<T>>::contains_key(order_id), Error::<T>::OfferNotFound);
     //get offer details
     let offer = <AfloatOffers<T>>::get(order_id).unwrap();
-	Ok(())
+	match offer.status {
+		OfferStatus::CREATED => {
+			<AfloatOffers<T>>::try_mutate(order_id, |offer| -> DispatchResult {
+				let offer = offer.as_mut().ok_or(Error::<T>::OfferNotFound)?;
+				offer.cancellation_date = Some(T::TimeProvider::now().as_secs());
+				offer.status = OfferStatus::CANCELLED;
+				Ok(())
+			})?;
+			Ok(())
+		}
+		_ => {
+			Err(Error::<T>::OfferTaken.into())
+		}
+	}
   }
 }

pallets/afloat/src/types.rs

@@ -56,7 +56,7 @@ pub enum SignUpArgs {
 }
 
 // ! Offer structures
-
+// ! The statuses with "TF" refer to the transfer form that is generated after a match is formed - so all of the TF_* statuses come after matched
 #[derive(
   Encode, Decode, Clone, Eq, PartialEq, RuntimeDebugNoBound, MaxEncodedLen, TypeInfo, Copy,
 )]
@@ -70,6 +70,7 @@ pub enum OfferStatus {
   TF_APPROVED,
   APPROVED,
   FILLED,
+  CANCELLED,
 }
 
 impl Default for OfferStatus {