fix: remove Identifier variant from few more enums

Author: Farhad-Shabani

ibc-apps/ics721-nft-transfer/types/src/error.rs

@@ -16,8 +16,6 @@ pub enum NftTransferError {
     ContextError(ContextError),
     /// decoding error: `{0}`
     Decoding(DecodingError),
-    /// identifier error: `{0}`
-    Identifier(IdentifierError),
     /// invalid trace: `{0}`
     InvalidTrace(String),
     /// invalid URI error: `{0}`
@@ -58,6 +56,12 @@ impl std::error::Error for NftTransferError {
     }
 }
 
+impl From<IdentifierError> for NftTransferError {
+    fn from(e: IdentifierError) -> Self {
+        Self::Decoding(DecodingError::Identifier(e))
+    }
+}
+
 impl From<Infallible> for NftTransferError {
     fn from(e: Infallible) -> Self {
         match e {}

ibc-core/ics02-client/types/src/error.rs

@@ -20,8 +20,6 @@ pub enum ClientError {
     Upgrade(UpgradeClientError),
     /// decoding error: `{0}`
     Decoding(DecodingError),
-    /// identifier error: `{0}`
-    Identifier(IdentifierError),
     /// invalid trust threshold: `{numerator}`/`{denominator}`
     InvalidTrustThreshold { numerator: u64, denominator: u64 },
     /// invalid client state type: `{0}`
@@ -110,7 +108,7 @@ impl From<DecodingError> for ClientError {
 
 impl From<IdentifierError> for ClientError {
     fn from(e: IdentifierError) -> Self {
-        Self::Identifier(e)
+        Self::Decoding(DecodingError::Identifier(e))
     }
 }
 
@@ -119,7 +117,6 @@ impl std::error::Error for ClientError {
     fn source(&self) -> Option<&(dyn std::error::Error + 'static)> {
         match &self {
             Self::FailedICS23Verification(e) => Some(e),
-            Self::Identifier(e) => Some(e),
             Self::Decoding(e) => Some(e),
             _ => None,
         }