Remove PLEASE REVIEW comments

CosmWasm · Aug 3, 2021 · 0ed75c9 · 0ed75c9
1 parent f1e1e4f
commit 0ed75c9
Show file tree

Hide file tree

Showing 5 changed files with 3 additions and 6 deletions.
diff --git a/packages/cw20/schema/cw20_execute_msg.json b/packages/cw20/schema/cw20_execute_msg.json
@@ -284,7 +284,7 @@
               ]
             },
             "project": {
-              "description": "A URL pointing to the project behind this token. PLEASE REVIEW: is this dangerous from a security point of view?",
+              "description": "A URL pointing to the project behind this token.",
               "type": [
                 "string",
                 "null"
@@ -395,7 +395,7 @@
       "description": "This is used for uploading logo data, or setting it in InstantiateData",
       "anyOf": [
         {
-          "description": "A reference to an externally hosted logo. Must be a valid HTTP or HTTPS URL. PLEASE REVIEW: is this dangerous from a security point of view?",
+          "description": "A reference to an externally hosted logo. Must be a valid HTTP or HTTPS URL.",
           "type": "object",
           "required": [
             "url"

diff --git a/packages/cw20/schema/marketing_info_response.json b/packages/cw20/schema/marketing_info_response.json
@@ -29,7 +29,7 @@
       ]
     },
     "project": {
-      "description": "A URL pointing to the project behind this token. PLEASE REVIEW: is this dangerous from a security point of view?",
+      "description": "A URL pointing to the project behind this token.",
       "type": [
         "string",
         "null"

diff --git a/packages/cw20/src/logo.rs b/packages/cw20/src/logo.rs
@@ -7,7 +7,6 @@ use serde::{Deserialize, Serialize};
 #[serde(rename_all = "snake_case")]
 pub enum Logo {
     /// A reference to an externally hosted logo. Must be a valid HTTP or HTTPS URL.
-    /// PLEASE REVIEW: is this dangerous from a security point of view?
     Url(String),
     /// Logo content stored on the blockchain. Enforce maximum size of 5KB on all variants
     Embedded(EmbeddedLogo),

diff --git a/packages/cw20/src/msg.rs b/packages/cw20/src/msg.rs
@@ -60,7 +60,6 @@ pub enum Cw20ExecuteMsg {
     /// Setting Some("") will clear this field on the contract storage
     UpdateMarketing {
         /// A URL pointing to the project behind this token.
-        /// PLEASE REVIEW: is this dangerous from a security point of view?
         project: Option<String>,
         /// A longer description of the token and it's utility. Designed for tooltips or such
         description: Option<String>,

diff --git a/packages/cw20/src/query.rs b/packages/cw20/src/query.rs
@@ -81,7 +81,6 @@ pub struct MinterResponse {
 #[derive(Serialize, Deserialize, Clone, PartialEq, JsonSchema, Debug)]
 pub struct MarketingInfoResponse {
     /// A URL pointing to the project behind this token.
-    /// PLEASE REVIEW: is this dangerous from a security point of view?
     pub project: Option<String>,
     /// A longer description of the token and it's utility. Designed for tooltips or such
     pub description: Option<String>,