Merge pull request #7100 from paritytech/jg-dapp-localUrl

Local dapp development URL

dapps/src/apps/app.rs

@@ -26,6 +26,8 @@ pub struct App {
 	pub author: String,
 	#[serde(rename="iconUrl")]
 	pub icon_url: String,
+	#[serde(rename="localUrl")]
+	pub local_url: Option<String>,
 }
 
 impl App {
@@ -38,6 +40,7 @@ impl App {
 			version: info.version.to_owned(),
 			author: info.author.to_owned(),
 			icon_url: info.icon_url.to_owned(),
+			local_url: info.local_url.to_owned(),
 		}
 	}
 }
@@ -50,6 +53,7 @@ impl Into<EndpointInfo> for App {
 			version: self.version,
 			author: self.author,
 			icon_url: self.icon_url,
+			local_url: self.local_url,
 		}
 	}
 }

dapps/src/apps/fetcher/mod.rs

@@ -306,6 +306,7 @@ mod tests {
 			version: "".into(),
 			author: "".into(),
 			icon_url: "".into(),
+			local_url: "".into(),
 		}, Default::default(), None);
 
 		// when

dapps/src/apps/fs.rs

@@ -56,6 +56,7 @@ fn read_manifest(name: &str, mut path: PathBuf) -> EndpointInfo {
 				version: "0.0.0".into(),
 				author: "?".into(),
 				icon_url: "icon.png".into(),
+				local_url: None,
 			}
 		})
 }

dapps/src/endpoint.rs

@@ -44,6 +44,7 @@ pub struct EndpointInfo {
 	pub version: String,
 	pub author: String,
 	pub icon_url: String,
+	pub local_url: Option<String>,
 }
 
 pub type Endpoints = BTreeMap<String, Box<Endpoint>>;

dapps/src/page/builtin.rs

@@ -132,6 +132,7 @@ impl From<Info> for EndpointInfo {
 			description: info.description.into(),
 			author: info.author.into(),
 			icon_url: info.icon_url.into(),
+			local_url: None,
 			version: info.version.into(),
 		}
 	}

js/package.json

@@ -158,6 +158,7 @@
     "lodash.uniq": "4.5.0",
     "material-ui": "0.16.5",
     "prop-types": "15.5.10",
+    "query-string": "5.0.1",
     "react": "15.6.1",
     "react-dom": "15.6.1",
     "react-intl": "2.1.5",

js/src/Dapp/dapp.js

@@ -110,7 +110,9 @@ export default class Dapp extends Component {
 
     switch (app.type) {
       case 'local':
-        src = `${dappsUrl}/${app.id}/`;
+        src = app.localUrl
+          ? `${app.localUrl}?appId=${app.id}`
+          : `${dappsUrl}/${app.id}/`;
         break;
 
       case 'network':

js/src/inject.js

@@ -15,16 +15,19 @@
 // along with Parity.  If not, see <http://www.gnu.org/licenses/>.
 
 import Api from '@parity/api';
+import qs from 'query-string';
 import Web3 from 'web3';
 
 import web3extensions from './web3.extensions';
 
 function initProvider () {
-  const parts = window.location.pathname.split('/');
-  let appId = parts[1];
+  const path = window.location.pathname.split('/');
+  const query = qs.parse(window.location.search);
+
+  let appId = path[1] || query.appId;
 
   if (appId === 'dapps') {
-    appId = parts[2];
+    appId = path[2];
   } else if (!Api.util.isHex(appId)) {
     appId = Api.util.sha3(appId);
   }

parity/dapps.rs

@@ -295,6 +295,7 @@ mod server {
 					version: app.version,
 					author: app.author,
 					icon_url: app.icon_url,
+					local_url: app.local_url,
 				})
 				.collect()
 		}

rpc/src/v1/types/dapps.rs

@@ -31,6 +31,9 @@ pub struct LocalDapp {
 	/// Dapp icon
 	#[serde(rename="iconUrl")]
 	pub icon_url: String,
+	/// Local development Url
+	#[serde(rename="localUrl")]
+	pub local_url: Option<String>,
 }
 
 #[cfg(test)]
@@ -40,7 +43,7 @@ mod tests {
 
 	#[test]
 	fn dapp_serialization() {
-		let s = r#"{"id":"skeleton","name":"Skeleton","description":"A skeleton dapp","version":"0.1","author":"Parity Technologies Ltd","iconUrl":"title.png"}"#;
+		let s = r#"{"id":"skeleton","name":"Skeleton","description":"A skeleton dapp","version":"0.1","author":"Parity Technologies Ltd","iconUrl":"title.png","localUrl":"http://localhost:5000"}"#;
 
 		let dapp = LocalDapp {
 			id: "skeleton".into(),
@@ -49,6 +52,7 @@ mod tests {
 			version: "0.1".into(),
 			author: "Parity Technologies Ltd".into(),
 			icon_url: "title.png".into(),
+			local_url: "http://localhost:5000".into(),
 		};
 
 		let serialized = serde_json::to_string(&dapp).unwrap();