fix: use GraphTally as EIP712 domain for TAP v2

Signed-off-by: Tomás Migone <[email protected]>

Author: tmigone

tap_core/src/lib.rs

@@ -56,7 +56,7 @@ impl TapVersion {
 }
 
 /// The domain separator is defined as:
-/// - `name`: "TAP"
+/// - `name`: "TAP" for V1, "GraphTally" for V2 - This could be a fn argument but we don't want to change the function signature
 /// - `version`: "1" or "2" depending on protocol version
 /// - `chain_id`: The chain ID of the chain where the domain separator is deployed.
 /// - `verifying_contract`: The address of the contract that is verifying the signature.
@@ -65,8 +65,13 @@ pub fn tap_eip712_domain(
     verifying_contract_address: Address,
     version: TapVersion,
 ) -> Eip712Domain {
+    let name = match version {
+        TapVersion::V1 => "TAP",
+        TapVersion::V2 => "GraphTally",
+    };
+
     eip712_domain! {
-        name: "TAP",
+        name: name,
         version: version.as_str(),
         chain_id: chain_id,
         verifying_contract: verifying_contract_address,