Skip to content

Commit 1be5fbe

Browse files
authored
Merge pull request #38 from oslabs-beta/execBranch
Code cleanup
2 parents 390f477 + 8c42c47 commit 1be5fbe

36 files changed

+690
-248
lines changed

Contributors.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,14 @@ Below is a list of features and improvements to which you can contribute. If you
1313
- Clean up unused dependencies from pre-4.0
1414
- Connecting to PostgreSQL database
1515
- When connecting to PostgreSQL to grab the data from the database, it takes a really long time to load (approximately 2 minutes); further investigate pg_dump alternatives for faster loading.
16+
- Integration and End-to-End testing for the codebase.
17+
- Live query feedback so users can see what their changes to the canvas look like as queries.
1618

1719
Known bugs/issues
1820

1921
- Table connection (handle) does not automatically render after a foreign key is added and the column data is saved.
22+
- In production, remote postgres database connections do not work. We suspect this is due to container environment (nodeJS) not having access to pg_dump command.
23+
- In development, WSL/Ubuntu users may have trouble running pg_dump due to version mismatch. Run `sudo apt-get --purge remove postgresql` to clear configuration files and zombie directories before installing postgresql fresh.
24+
- Undo function will not regress the state of the canvas to the point where it is empty. Whether this is a design choice or an issue is up to you.
25+
2026

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
FROM node:16.17.0
22
WORKDIR /usr/src/app
33
COPY . /usr/src/app
4-
RUN npm ci && npm cache clean --force && npm run build
4+
RUN npm ci && npm cache clean --force && npm run build
55
EXPOSE 3000
66
ENTRYPOINT npm run start

README.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -56,19 +56,21 @@
5656
---
5757

5858
### Getting started
59-
59+
You will need your own mySQL and redis databases for backend functions, and Google Cloud project for OAuth.
6060
- Fork and clone this repo
61-
- Add a .env file to the root directory
6261
- Add a db_schemas folder in server directory
62+
- Add a .env file to the root directory
6363
- Go to the Google Cloud Platform Console
64-
- Set up OAuth 2.0 credentials as laid out in <a href="https://support.google.com/cloud/answer/6158849?hl=en">here</a>
64+
- Set up OAuth 2.0 credentials as laid out in <a href="https://support.google.com/cloud/answer/6158849?hl=en">here</a>. Make sure
65+
your authorized redirect URI in the credentials you create matches the GOOGLE_AUTH_CALLBACK variable below.
6566
- Populate the .env file with the newly created code below:
6667

6768
```bash
6869
GOOGLE_AUTH_CALLBACK = ''
6970
DATABASE_URL = [mysql url for storing user data]
7071
DEV_SERVER_ENDPOINT = 'http://localhost:3000'
7172
DEV_CLIENT_ENDPOINT = 'http://localhost:8080'
73+
CLIENT_ENDPOINT = 'http://localhost:3000'
7274
GOOGLE_AUTH_CALLBACK = 'http://localhost:3000/api/oauth/google'
7375
GOOGLE_AUTH_CLIENT_ID = [google auth client id]
7476
GOOGLE_AUTH_CLIENT_SECRET = [google auth client secret]
@@ -205,4 +207,4 @@ npm run cert
205207

206208
### License
207209

208-
dbSpy is developed under the MIT license. See more about [dbSpy4.0 updates here](https://github.com/oslabs-beta/dbSpy/releases/tag/v4.0.0).
210+
dbSpy is developed under the MIT license. See more about dbSpy4.0 updates [here](https://github.com/oslabs-beta/dbSpy/releases/tag/v4.0.0).

0 commit comments

Comments
 (0)