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.

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

Merge develop into main #608

Open
wants to merge 120 commits into
base: main
Choose a base branch
from
Open

Merge develop into main #608

wants to merge 120 commits into from

Conversation

LoTerence
Copy link
Member

@LoTerence LoTerence commented Nov 7, 2024

Fixes #610

This is going to break a lot of things so I need to test it carefully.

This will break the current staging deployment - https://stage.civictechjobs.org/

Actions

We need to test this in its own AWS deployment before merging into main.

  • Deploy this branch (develop) to AWS
  • Fully test the new deployment

Tests:

  • Frontend is functional
  • API server is functional
  • admin is functional
  • AWS deployment configuration is correct

Changes

  • Biggest change: Total reset of backend
  • initial vite migration
  • eslint + prettier added
  • new python linters added
  • mkdocs deployment revisions
  • docker is set up to adapt to new changes in backend and vite
  • enabled django admin
  • Updated docker stage environment build
  • prepared develop branch for staging deployment

bzzz-coding and others added 30 commits February 28, 2024 16:59
* Refactored QualifiersPageRoles, added context, and updated documentataion on COP JSON structure

* Removed copData fetching from QualifiersPageRoles and lifted copData to QualifiersContext

* Cleaned up strings for JSON

* Removed extra backtick in markdown
* Added vite, necessary plugins, typescript configs

- svgr to create react components from svg files
- tsconfigpaths for relative routing in typescript
- createHtmlPlugin to generate html template

* Renamed all js files to ts for compiling

- Contains errors in ts that needs to be addressed

* example html for vite

* utilize svgr for vite plugin due to no babel

* updated index file to utilize vite env

* Changed to absolute paths

Vite will not recognize relative paths for scss files
used @ as an alias for src directory

* attempt at vite build

* Removed warning for improperly cased props

* Added aria label for react-axe accessibility warning
* Added healthcheck endpoint to API

* Imported app version from settings and upgraded node in jest test action

* Updated setup-node

* Updated node-version
* Update intro.md (#512)

Hey updated a few steps for onboarding under the intro.

* init sql file to create post table

* Updated docker to create postgres database

- updated git ignore to include /data directory

* Updated dev.env example

- found bug that prevented postgres from connecting

---------

Co-authored-by: kchotani <87041811+kchotani@users.noreply.github.com>
* Pgdb connected

- added configs in django.settings
- installed psycopg2-binary **for development only**
- psycopg2 for production requires certain system depdencies

* init Opportunities model

- Recreated from previous PR
- Renamed post model to "opportunities"
- did migration

* Load sample data into database

- Added fixtures
- make sure to 'python manage.py loaddata ctjbackend/fixtures/*.json'
@LoTerence LoTerence changed the title Develop into main Merge develop into main Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge develop branch into main branch
6 participants