Skip to content

Commit

Permalink
Merge pull request #124 from oslabs-beta/main
Browse files Browse the repository at this point in the history
Edits
  • Loading branch information
choukevin612 authored Feb 8, 2024
2 parents 031e40a + 797a808 commit c5dcab0
Show file tree
Hide file tree
Showing 9 changed files with 24 additions and 29 deletions.
2 changes: 1 addition & 1 deletion DEV_README.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ There are still some filesystem read/write calls in the front end. This should b
<p><b> WHAT IS BROKEN: </b></p>

<p>1. The application on Windows may periodically crash.</p>
<p>2. There are import issues on mac computers.</p>
<p>2. There are import issues on Mac computers.</p>

- Unable to import pg or mySQL database files
<p>3. Duplicates appear on previous queries.</p>
Expand Down
14 changes: 6 additions & 8 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -70,19 +70,18 @@ To get started on contributing and editing databases to this project:

<div align="center">
<img src="./assets/readmeImages/gifs/login.gif" width=800/>
<!-- <img src="./assets/readmeImages/gifs/quick_start.gif" width=800/> -->
<img src="./assets/readmeImages/gifs/importCreateDB.gif" width=800/>
</div>

- Databases

- In the 'DATABASES' view, an interactive Entity Relationship Diagram (`ER DIAGRAM`) is displayed for the selected database.
- Users can now save table layout in version 13.
<br>

<br>
<div align="center">
<img src="./assets/readmeImages/gifs/save_db_layout.gif" width=800/>
</div>
<br>
<div align="center">
<img src="./assets/readmeImages/gifs/save_db_layout.gif" width=800/>
</div>


- Users can select `TABLE` to see selected database in tabular form.
Expand Down Expand Up @@ -135,8 +134,7 @@ To get started on contributing and editing databases to this project:

<br>
<div align="center">
<img src="./assets/readmeImages/gifs/importCreateDB.gif" width=800/>
<img src="./assets/readmeImages/gifs/modify_db.gif" width=800/>
<img src="./assets/readmeImages/gifs/export.gif" width=800/>
</div>

- Queries
Expand Down
Binary file added assets/readmeImages/gifs/dataTypeChange.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/readmeImages/gifs/export.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/readmeImages/gifs/importCreateDB.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/readmeImages/gifs/login.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/readmeImages/gifs/querySearch.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added assets/readmeImages/gifs/save_db_layout.gif
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
37 changes: 17 additions & 20 deletions roadmap.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,42 +8,44 @@

- Refactor codebase to elimiate <strong>prop-drilling</strong> allowing for future iterability.

- Update ui of the initial landing page of application with cloud database instructions
- Continue working on state management (usereducer, usecontext, etc)

- continue working on state management (usereducer, usecontext, etc)
- Update UI of the initial landing page of application with cloud database instructions

- fix outdated dependencies, currently packages must be force installed
- Fix outdated dependencies, currently packages must be force installed

- Mac vs Windows:

- In the '../backend/src/models/configModel.ts' on line 9 - 11 specifies where the configFile will be downloaded. The 'home' variable will be different for Mac and Windows users. Utilize an if conditional statement here to prevent the need for maintaining two branches (main and devosx) for windows and mac.

- For Windows set `home = process.cwd()` on line 10 and for Mac set <br>`home = ${os.homedir()}/Documents/SeeQR` </br>

- Database management

- currently postgres imports/duplicates only works for either windows or osx/linux. main branch supports windows, devosx supports mac/linux, resolve this asap to not have to keep up with two branches
- Currently postgres imports/duplicates only works for either windows.

- perhaps use .pgpass file instead of pgpassword environment variable for security (for postgresql imports/duplicates)
- Perhaps use .pgpass file instead of pgpassword environment variable for security (for postgresql imports/duplicates)

- found in backend/helperFunctions.ts
- Found in backend/helperFunctions.ts

- Fix importing MySQL databases (currently creating an extra hollow copy or not working at all)

- Currently importing MySQL databases will work when only MySQL server is up on MAC OS.

- MySQL databases that are imported will only show data type, but not column name.

- add ability to have multiple sqlite databases

- fix deleting sqlite databases (currently only works randomly, likely not closing connection before attempting unlink)
- Fix sqlite and RDS databases (currently only Postgres and MySQL are working)

- support for amazon aurora (beware of billing)
- Support for amazon aurora (beware of billing)


- Queries page

- Fix query execution plan table view, likely broke while updating frontend dependencies
- Fix query execution plan table view, likely broke while updating frontend dependencies

- Utilize local storage to save query history. Currently the history disappears when we reload application.

- Work on explain function for mysql and sqlite, may have different metadata from existing postgres implementation, display whatever you can get
- Work on explain function for mysql and sqlite, may have different metadata from existing postgres implementation, display whatever you can get


- 2D visualization / ER tables:

Expand All @@ -65,11 +67,11 @@

- resolve issue of creating additional columns for each constraint (mostly mysql)

- rds pg cloud queries seem to be creating the tables in more than just the selected database.
- rds pg cloud queries not set up

- rds my sql cloud queries wont let you create multiple tables at once. as in you have to create one table, then make another query to make your second table.

- when you create a new cloud pg database, it seems to have all the other databases tables aswell.
- when you create a new cloud pg database, it seems to have all the other databases tables as well.


- 3D visualization:
Expand All @@ -86,9 +88,4 @@

- Add VR functionality?

- Mac vs Windows:

- In the '../backend/src/models/configModel.ts' on line 9 - 11 specifies where the configFile will be downloaded. The 'home' variable will be different for Mac and Windows users. Utilize an if conditional statement here to prevent the need for maintaining two branches (main and devosx) for windows and mac.

- For Windows set `home = process.cwd()` on line 10 and for Mac set <br>`home = ${os.homedir()}/Documents/SeeQR` </br>

0 comments on commit c5dcab0

Please sign in to comment.