Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

New React Frontend #33

Closed
wants to merge 62 commits into from
Closed

New React Frontend #33

wants to merge 62 commits into from

Conversation

Yasamato
Copy link
Collaborator

@Yasamato Yasamato commented Apr 16, 2021 •
edited
Loading

replaces #25

flask backend

  • move to SQLAlchemy
  • switch to usage via mariaDB/postgress
  • reimplement table data manipulation
  • add table api endpoints
  • add tab api endpoints
  • add column api endpoints
  • handling data manipulation on column changes

react frontend

  • create react project
  • add build instructions
  • Editor:
    • reimplement table data editor
    • add table editor
    • add tab editor
    • add column editor
  • Pages:
    • add admin view for table, tab and column editor
    • reimplement table view
    • reimplement tab view
    • reimplement navbar
    • reimplement help page
  • think about things we should consider regarding SEO

@Yasamato
Prepare backend api for tab edits
f4fec4b 
This is currently a breaking build and WIP
@Yasamato
Reorder Dockerfile for better build cache
83d6a66
@Yasamato
Switched to SQLAlchemy
8ecc665 
Simply because it is just way more powerful than dataset
@Yasamato
Correct api endpoints
b24ec62
@Yasamato
Update static/js/utils.js
8c8f3b0
@Yasamato
Update static/js/utils.js
7c9c8c5
@Yasamato
Cleanup misc
3cab556
@Yasamato
Return IDs instead of whole objects
55c53c9
@Yasamato
Log errors only to console
4882cc2
@Yasamato
Adjust .gitignore
a7c7598
@Yasamato
Flutter init
faa07b2
@Yasamato
Cleanup ignore files and Dockerfile
fa98b82
@Yasamato
Fix docker build
40d1dcb
@Yasamato
Misc. code cleanup
77c4e2d
@Yasamato
Adjust github issue templates
e81360c
@Yasamato
Add allow CORS to /api fetch only routes
5edde27
@Yasamato
Adjust Dart version
4ef8176
@Yasamato
Only run github action on push
173a390 
No need to run the github action twice on active PRs
@Yasamato
Update queries.py
1a7e4b7
@Yasamato
Add inital fetch layouts
f6b11b1
@Yasamato
Sticky header
bfa9561
@Yasamato
Adjust API to respond recursivly
bde812d
@Yasamato
Drop flutter, replace with React
a6a2ed5
@Yasamato
Update src/App.tsx
2aee2e0
@Yasamato
Update src/App.tsx
9e9fa6a
@Yasamato
Update src/App.tsx
3c7c8cf
@Yasamato
Update src/index.tsx
ce25c6f
@Yasamato
Update src/index.tsx
4388f86
@Yasamato
Correctly bind callbacks
408e4d2
@Yasamato
Update src/App.tsx
409c4e7
@lgtm-com
Copy link

lgtm-com bot commented Apr 17, 2021

This pull request introduces 3 alerts and fixes 1 when merging 81ada01 into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 17, 2021

This pull request introduces 3 alerts and fixes 1 when merging f854f9b into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 17, 2021

This pull request introduces 4 alerts and fixes 1 when merging 1d35adc into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 18, 2021

This pull request introduces 4 alerts and fixes 1 when merging cbb493d into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@Yasamato
Update API documentation
82e332f 
TODO: document edit api
@lgtm-com
Copy link

lgtm-com bot commented Apr 18, 2021

This pull request introduces 4 alerts and fixes 1 when merging 82e332f into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 18, 2021

This pull request introduces 3 alerts and fixes 1 when merging ae27c71 into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 18, 2021

This pull request introduces 3 alerts and fixes 1 when merging 8e55870 into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 18, 2021

This pull request introduces 3 alerts and fixes 1 when merging a44c74c into a4db746 - view on LGTM.com

new alerts:

  • 1 for Direct state mutation
  • 1 for Potentially inconsistent state update
  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

@lgtm-com
Copy link

lgtm-com bot commented Apr 18, 2021

This pull request introduces 1 alert and fixes 1 when merging 73de3f8 into a4db746 - view on LGTM.com

new alerts:

  • 1 for Unused or undefined state property

fixed alerts:

  • 1 for Inefficient regular expression

Emiliaaah and others added 2 commits April 19, 2021 15:32
@Emiliaaah
External Database support (#34)
6636427 
* Added dependencies postgres & mysql

* import db connection from env variable

* Added varchar character limit for mysql support

* Added mysql & postgres dependencies for psycopg2

* Added parameter for external db connections

* Manifest was already defined

* db check before init script runs

* Removed db file check

* Added logging
@Yasamato
Update TableView.tsx
c3f8b29
@Yasamato
Copy link
Collaborator Author

Yasamato commented Aug 6, 2021

Closed in favor of #39

@Yasamato Yasamato closed this Aug 6, 2021
@Yasamato Yasamato deleted the dev branch August 6, 2021 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Yasamato @Emiliaaah