generated from phucvinh57/fastify-template
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Bugfix/access config db before db is running #23
Merged
quannhg
merged 47 commits into
test/staging
from
bugfix/access-config-db-before-db-is-running
Nov 18, 2023
Merged
Bugfix/access config db before db is running #23
quannhg
merged 47 commits into
test/staging
from
bugfix/access-config-db-before-db-is-running
Nov 18, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ci(commit-lint): add ci check conventional commit message
…hen starting server
…ure/implement-authorization-with-admin-and-student-roles
Include admin, student and *. Default is admin.
using minio to store image and genenrate url
…ing ssps external network also update sign up handler with new atribute of user
Feat/implement home
…with-admin-and-student-roles Feature/implement authorization with admin and student roles
Feature/auth
api work well but swagger of it haven't yet, also implement a portion of execute printing request api
…nism for add new file Seperate file to new table to keep more information. Remove file in Minio if update file information to database fail.
…rint whole printing request node thermal printer can not use so i change to node-printer, this package not update from 2021 so i use the fork of thiagoelg. Also i change the logic to print whole printing request instead of one file. In addition, i write printing interface for easy maintain and extended. BREAKING CHANGE: change from node-thermal-printer to node-printer
…handler to accommodate changes Refactor the database schema by creating dedicated tables for locations, enhancing data organization and improving flexible. Update corresponding data handler to seamlessly work with these changes, ensuring compatibility with the new database structure. This modification optimizes data retrieval, simplifies future updates, and enhances overall system performance.
…s for printing request
…t stage of Dockerfile (release.yml) Also make sure specific folder for deploy is created
(docker-compose.yml) adjust expose port of fastify (release.yml) add CORS_WHITE_LIST to .env
…action Test/implement cd using GitHub action
Feature/implement upload file api
also change attachFieldToBody of fastify/multipart to true for get other information of file instead of just buffer
…te apis Another change: Add atribute for file, result of upload api. Hash name of file to store in minio.
also adjust get all and delete file api
Feature/split upload api
…pment feat(workflows): split cd for deploy of development and product
Implement APIs, modify cd with new data in .env.
feat(purchase coin): implement PayPal Coin Purchase APIs
also let dev cd generate seed data
quick fix to remain developer process, need implement logic for developer and production server in future
…velop Hotfix/accept-all-request-on-develop
…atabase Save value of DOLLAR_TO_COIN in database make dotenv cannot read any env variable. Temporary save it's value in code to quick fix.
fix(configuration): save value of DOLLAR_TO_COIN in code instead of database
…g Google Oauth Previously, encountered an error when accessing env variables from `envs` outside `env.ts` in the envalid library. As a temporary solution, switched to using `process.env`. Additionally, started implementing the basics of Google OAuth but haven't verified it yet. BREAKING CHANGE: Changing to using process.env instead of envs (envalid). Need to fix later.
Enable dynamic environment configuration for Google Cloud environments, resolving an issue where environment variables were not properly utilized. However, note that this enhancement does not yet extend to Minio and Node environments, where environment variables remain ineffective.
…utils, and repository modules Also add transaction to createStudent in auth handler. Add neccesary env to cd of dev and product server.
Featute/integrate google oauth
The older version return {id, username}. But after integrate google oauth, some user may don't have username and is identify by their email, so return {id, email} for better semantically accurate representation.
…y-id fix(user.handler.ts): change return of api/user to {id, email}
…der to handlers avoid circular dependency
… development deployments Implement CD for cloned containers to validate successful development deployments. On an open pull request to the staging branch, deploy a clone of the intended container to the development server for validation. On a closed pull request, remove all test deployment containers, images, and folders.
…ervice and developer service Refactor the project to use distinct ports and Docker-compose files for the test deployment service and the developer service. Introduce separate naming conventions and utilize new environment variables for improved clarity and maintainability.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.