Skip to content

Commit

Permalink
Revert "V13 launch"
Browse files Browse the repository at this point in the history
  • Loading branch information
choukevin612 authored Feb 8, 2024
1 parent 277c728 commit c8f5375
Show file tree
Hide file tree
Showing 7 changed files with 174 additions and 26,113 deletions.
2 changes: 0 additions & 2 deletions DEV_README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,6 @@ Run npm run dev twice if you do not manually run tsc to compile the files first.

- Migrated ERTabling to tableTabBar component to access the ERD because it lacked a parent compartment for prop drilling, hindering the addition of new features. Going forward, a more maintainable solution should be implemented like Redux or Zustand.

- Migrated ERTabling to tableTabBar component to access the ERD because it lacked a parent compartment for prop drilling, hindering the addition of new features. Going forward, a more maintainable solution should be implemented like Redux or Zustand.

<p><b>2. Isolating Database</b> <br> One of the biggest tasks that we tried but <b>did not finish</b> is isolating the concerns of each database type (DBType). The current application has multiple</p>
<code>if (database === DBType.postgres) {}<br>
else if (database === DBType.mysql) {}<br>
Expand Down
2 changes: 1 addition & 1 deletion __tests__/frontend/lib/unittesting.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ describe('runSelectAllQuery', () => {
{ sqlString, selectedDb },
curDBType,
);

console.log(result);

expect(mockConnectionModel.connectToDB).toHaveBeenCalledWith('test', 'pg');
expect(mockConnectionModel.connectToDB).toHaveBeenCalledTimes(1);
Expand Down
1 change: 1 addition & 0 deletions backend/src/ipcHandlers/handlers/queryHandler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -264,6 +264,7 @@ export async function runSelectAllQuery(event, {sqlString, selectedDb}:SelectAll
try {
await connectionModel.connectToDB(selectedDb, curDBType);
const results = await queryModel.query(sqlString, [], curDBType);
console.log('good',results.rows)
return results?.rows
} catch (error) {
console.log(error, 'in runSelectAllQuery')
Expand Down
1 change: 1 addition & 0 deletions backend/src/utils/dummyData/dummyDataMain.ts
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ export const getRandomInt = (min: number, max: number) => {
// helper function to generate random data based on a given column's data type
const generateDataByType = (columnObj: ColumnObj): string | number => {
let length;
console.log(columnObj.data_type);
// updated the new faker package so updated to follow proper documentation.
switch (columnObj.data_type) {
case 'smallint':
Expand Down
Loading

0 comments on commit c8f5375

Please sign in to comment.