Skip to content

Commit bf4c6d0

Browse files
committed
Extract config.js from common.js to make client setting file clean and also make upgrade easier
1 parent 917537d commit bf4c6d0

9 files changed

Lines changed: 21 additions & 17 deletions

File tree

.gitignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,5 +19,5 @@ backups/
1919

2020
# ignore config files
2121
config.json
22-
public/js/common.js
22+
public/js/config.js
2323
.sequelizerc

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,10 @@ There are some configs you need to change in the files below
8383

8484
```
8585
./config.json --- for server settings
86-
./public/js/common.js --- for client settings
86+
./public/js/config.js --- for client settings
8787
```
8888

89-
Client settings `common.js`
89+
Client settings `config.js`
9090
---
9191

9292
| variables | example values | description |
@@ -148,7 +148,7 @@ Third-party integration api key settings
148148
| ------- | --------- | ----------- |
149149
| facebook, twitter, github, gitlab, dropbox, google | `config.json` | for signin |
150150
| imgur | `config.json` | for image upload |
151-
| google drive, dropbox | `public/js/common.js` | for export and import |
151+
| google drive, dropbox | `public/js/config.js` | for export and import |
152152

153153
Third-party integration oauth callback urls
154154
---

bin/setup

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ if [ ! -f config.json ]; then
2525
cp config.json.example config.json
2626
fi
2727

28-
if [ ! -f publis/js/common.js ]; then
29-
cp public/js/common.js.example public/js/common.js
28+
if [ ! -f publis/js/config.js ]; then
29+
cp public/js/config.js.example public/js/config.js
3030
fi
3131

3232
if [ ! -f .sequelizerc ]; then
@@ -43,7 +43,7 @@ Edit the following config file to setup hackmd server and client.
4343
Read more info at https://github.com/hackmdio/hackmd#configuration-files
4444
4545
* config.json -- server config
46-
* public/js/common.js -- client config
46+
* public/js/config.js -- client config
4747
* .sequelizerc -- db config
4848
4949
EOF
Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,4 @@
11
//common
2-
var domain = ''; // domain name
3-
var urlpath = ''; // sub url path, like: www.example.com/<urlpath>
4-
//settings
5-
var debug = false;
6-
7-
var GOOGLE_API_KEY = '';
8-
var GOOGLE_CLIENT_ID = '';
9-
10-
var DROPBOX_APP_KEY = '';
11-
122
var port = window.location.port;
133
var serverurl = window.location.protocol + '//' + (domain ? domain : window.location.hostname) + (port ? ':' + port : '') + (urlpath ? '/' + urlpath : '');
144
var noteid = urlpath ? window.location.pathname.slice(urlpath.length + 1, window.location.pathname.length).split('/')[1] : window.location.pathname.split('/')[1];

public/js/config.js.example

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
//config
2+
var domain = ''; // domain name
3+
var urlpath = ''; // sub url path, like: www.example.com/<urlpath>
4+
//settings
5+
var debug = false;
6+
7+
var GOOGLE_API_KEY = '';
8+
var GOOGLE_CLIENT_ID = '';
9+
10+
var DROPBOX_APP_KEY = '';

public/views/foot.ejs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@
7575
<script src="<%- url %>/vendor/keymaster/keymaster.js" defer></script>
7676
<script src="<%- url %>/js/google-drive-upload.js" defer></script>
7777
<script src="<%- url %>/js/google-drive-picker.js" defer></script>
78+
<script src="<%- url %>/js/config.js" defer></script>
7879
<script src="<%- url %>/js/common.js" defer></script>
7980
<script src="<%- url %>/js/extra.js" defer></script>
8081
<script src="<%- url %>/js/render.js" defer></script>

public/views/index.ejs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -205,6 +205,7 @@
205205
<script src="<%- url %>/vendor/file-saver/FileSaver.min.js" defer></script>
206206
<script src="<%- url %>/vendor/store-js/store.min.js" defer></script>
207207
<script src="<%- url %>/vendor/lz-string/libs/lz-string.min.js" defer></script>
208+
<script src="<%- url %>/js/config.js" defer></script>
208209
<script src="<%- url %>/js/common.js" defer></script>
209210
<script src="<%- url %>/js/history.js" defer></script>
210211
<script src="<%- url %>/js/cover.js" defer></script>

public/views/pretty.ejs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,7 @@
127127
<script src="<%- url %>/vendor/viz.js/viz.js" defer></script>
128128
<script src="<%- url %>/vendor/pdfobject/pdfobject.min.js" defer></script>
129129
<script src="<%- url %>/vendor/md-toc.js" defer></script>
130+
<script src="<%- url %>/js/config.js" defer></script>
130131
<script src="<%- url %>/js/common.js" defer></script>
131132
<script src="<%- url %>/js/extra.js" defer></script>
132133
<script src="<%- url %>/js/render.js" defer></script>

public/views/slide.ejs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,7 @@
122122
<script src="<%- url %>/vendor/viz.js/viz.js" defer></script>
123123
<script src="<%- url %>/vendor/pdfobject/pdfobject.min.js" defer></script>
124124
<script src="<%- url %>/vendor/md-toc.js" defer></script>
125+
<script src="<%- url %>/js/config.js" defer></script>
125126
<script src="<%- url %>/js/common.js" defer></script>
126127
<script src="<%- url %>/js/extra.js" defer></script>
127128
<script src="<%- url %>/js/render.js" defer></script>

0 commit comments

Comments
 (0)