Skip to content

Commit

Permalink
misc: remove querystring in http/Client (#236)
Browse files Browse the repository at this point in the history
* misc: remove querystring in http/Client

* fix: demo path issue

* chore: update lock file
  • Loading branch information
embbnux authored Oct 25, 2023
1 parent 687bd81 commit 7818d41
Show file tree
Hide file tree
Showing 32 changed files with 124 additions and 117 deletions.
2 changes: 0 additions & 2 deletions api-test/.npmrc

This file was deleted.

2 changes: 0 additions & 2 deletions demo/.npmrc

This file was deleted.

21 changes: 21 additions & 0 deletions demo/link-path.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
const fs = require('fs-extra');
const path = require('path');

if (!fs.existsSync(path.resolve(__dirname, './dist'))) {
fs.mkdirSync(path.resolve(__dirname, './dist'));
}

fs.ensureSymlinkSync(
path.resolve(__dirname, '../sdk/dist'),
path.resolve(__dirname, './dist/sdk')
);

fs.ensureSymlinkSync(
path.resolve(__dirname, '../subscriptions-deprecated/dist'),
path.resolve(__dirname, './dist/subscriptions-deprecated')
);

fs.ensureSymlinkSync(
path.resolve(__dirname, '../subscriptions/dist'),
path.resolve(__dirname, './dist/subscriptions')
);
7 changes: 5 additions & 2 deletions demo/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,10 +3,13 @@
"name": "@ringcentral-demo/sdk",
"private": true,
"scripts": {
"start": "http-server -p $DEMO_PORT"
"start": "npm run link-files && npm run server",
"server": "http-server -p $DEMO_PORT",
"link-files": "node ./link-path.js"
},
"devDependencies": {
"http-server": "^14.1.1"
"http-server": "^14.1.1",
"fs-extra": "^11.1.1"
},
"dependencies": {
"@ringcentral/sdk": "*",
Expand Down
10 changes: 5 additions & 5 deletions demo/src/apiKey.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -74,9 +74,9 @@ <h1>

</div>

<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down
14 changes: 7 additions & 7 deletions demo/src/backbone.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -34,11 +34,11 @@ <h1>

</div>

<script type="text/javascript" src="../node_modules/underscore/underscore.js"></script>
<script type="text/javascript" src="../node_modules/backbone/backbone.js"></script>
<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/underscore@1.13.6/underscore.js"></script>
<script type="text/javascript" src="https://unpkg.com/backbone@1.5.0/backbone.js"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down
14 changes: 7 additions & 7 deletions demo/src/core.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -37,10 +37,10 @@ <h1>
<script type="text/javascript">
window.fetch = null; // force to use polyfill
</script>
<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script src="../node_modules/@ringcentral/subscriptions/dist/ringcentral-subscriptions.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="../dist/subscriptions/ringcentral-subscriptions.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down Expand Up @@ -71,7 +71,7 @@ <h1>

function getPresence(cb) {

platform.get('/restapi/v1.0/account/~/extension/~/presence', {query: {detailedTelephonyState: true}})
platform.get('/restapi/v1.0/account/~/extension/~/presence', {detailedTelephonyState: true})
.then(function(response) {
return response.json();
}).then(cb).catch(function(e) {
Expand Down
10 changes: 5 additions & 5 deletions demo/src/facebook.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -85,9 +85,9 @@ <h1>
<script type="text/javascript">
window.fetch = null; // force to use polyfill
</script>
<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down
11 changes: 5 additions & 6 deletions demo/src/fax.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -73,10 +73,9 @@ <h1>
<script type="text/javascript">
window.fetch = null; // force to use polyfill
</script>
<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script type="text/javascript" src="../node_modules/pubnub/dist/web/pubnub.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/[email protected]/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down
4 changes: 2 additions & 2 deletions demo/src/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>JavaScript SDK &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down
10 changes: 5 additions & 5 deletions demo/src/login.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -106,9 +106,9 @@ <h1>
<script type="text/javascript">
window.fetch = null; // force to use polyfill
</script>
<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down
10 changes: 5 additions & 5 deletions demo/src/mms.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>Core &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -74,9 +74,9 @@ <h1>
<script type="text/javascript">
window.fetch = null; // force to use polyfill
</script>
<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script src="scripts.js" type="text/javascript"></script>
<script type="text/javascript">

Expand Down
2 changes: 1 addition & 1 deletion demo/src/oauth/redirect.html
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
<title>3-legged OAuth Redirect Handler &mdash; RingCentral&trade; Platform</title>
</head>
<body>
<script src="../../node_modules/@ringcentral/sdk/dist/ringcentral.js" type="text/javascript"></script>
<script src="../../dist/sdk/ringcentral.js" type="text/javascript"></script>
<script type="text/javascript">

RingCentral.SDK.handleLoginRedirect();
Expand Down
12 changes: 6 additions & 6 deletions demo/src/requirejs.html
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@
<html>
<head>
<title>RequireJS &mdash; RingCentral&trade; Platform</title>
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap.css">
<link rel="stylesheet" href="../node_modules/bootstrap/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap.css">
<link rel="stylesheet" href="https://unpkg.com/bootstrap@3.4.1/dist/css/bootstrap-theme.css">
<link rel="stylesheet" href="common.css">
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no">
<meta charset="UTF-8">
Expand Down Expand Up @@ -31,16 +31,16 @@ <h4>RingCentral JS SDK Properties</h4>

</div>

<script type="text/javascript" src="../node_modules/whatwg-fetch/dist/fetch.umd.js"></script>
<script type="text/javascript" src="../node_modules/es6-promise/dist/es6-promise.auto.js"></script>
<script src="../node_modules/requirejs/require.js" type="text/javascript"></script>
<script type="text/javascript" src="https://unpkg.com/whatwg-fetch@3.6.19/dist/fetch.umd.js"></script>
<script type="text/javascript" src="https://unpkg.com/es6-promise@latest/dist/es6-promise.auto.js"></script>
<script src="https://unpkg.com/requirejs@2.3.6/require.js" type="text/javascript"></script>
<script type="text/javascript">

//window.fetch = null;

requirejs.config({
paths: {
'ringcentral': '../node_modules/@ringcentral/sdk/dist/ringcentral',
'ringcentral': '../dist/sdk/ringcentral',
}
});

Expand Down
Loading

0 comments on commit 7818d41

Please sign in to comment.