apply cr changes

packages/contracts-periphery/contracts/universal/op-nft/OptimistInviter.sol

@@ -52,13 +52,13 @@ contract OptimistInviter is Semver, EIP712Upgradeable {
     /**
      * @notice Attestation key for granting invites.
      */
-    bytes32 public constant CAN_INVITE_ATTESTATION_KEY = bytes32("optimist.can-invite");
+    bytes32 public constant CAN_INVITE_ATTESTATION_KEY = keccak256("optimist.can-invite");
 
     /**
      * @notice Attestation key allowing the attested account to mint.
      */
     bytes32 public constant CAN_MINT_FROM_INVITE_ATTESTATION_KEY =
-        bytes32("optimist.can-mint-from-invite");
+        keccak256("optimist.can-mint-from-invite");
 
     /**
      * @notice Granter who can set accounts' invite counts.
@@ -249,6 +249,12 @@ contract OptimistInviter is Semver, EIP712Upgradeable {
         // Failing this check means that the issuer has used up all of their existing invites.
         require(count > 0, "OptimistInviter: issuer has no invites");
 
+        // Reduce the issuer's invite count by 1 by re-attesting the optimist.can-invite attestation
+        // with the new count. Can be unchecked because we check that the count is > 0 above.
+        unchecked {
+            --count;
+        }
+
         // Create the attestation that the claimer can mint from the issuer's invite.
         // The invite issuer is included in the data of the attestation.
         ATTESTATION_STATION.attest(
@@ -257,12 +263,6 @@ contract OptimistInviter is Semver, EIP712Upgradeable {
             abi.encode(_claimableInvite.issuer)
         );
 
-        // Reduce the issuer's invite count by 1 by re-attesting the optimist.can-invite attestation
-        // with the new count. Can be unchecked because we check that the count is > 0 above.
-        unchecked {
-            --count;
-        }
-
         ATTESTATION_STATION.attest(
             _claimableInvite.issuer,
             CAN_INVITE_ATTESTATION_KEY,