-
Notifications
You must be signed in to change notification settings - Fork 0
Cu 86a47rg9a/opt create plan flow/frank licona #30
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
Open
3FE3LE
wants to merge
39
commits into
main
Choose a base branch
from
CU-86a47rg9a/OPT-Create-Plan-Flow/Frank-Licona
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
3f2cf77
fix(opt): :bug: apply class transformation on date
3FE3LE 0aa33d8
feat(opt): :sparkles: add userId validation
3FE3LE 9b044ad
feat(auth): :sparkles: expiration token validation added & define new…
3FE3LE 4d11099
feat(opt): :passport_control: add guard on get endpoints to get user …
3FE3LE 63cd949
feat(opt): :card_file_box: generate new migration with new relations
3FE3LE 6a0f19f
fix(server): :bug: fix nest cli bug
3FE3LE c8fb381
feat(opt): :sparkles: add budget, activity & destination resources to…
3FE3LE f1b41ec
feat(opt): :sparkles: add destinies endpoint
3FE3LE b4bfada
feat(opt): :card_file_box: migration delete postal code & add country…
3FE3LE 6d1eac4
perf(global): :zap: refine update-dtos using PartialType
3FE3LE 927afa3
feat(opt): :card_file_box: migration add validations
3FE3LE eed8dec
feat(opt): :sparkles: add activity endpoints
3FE3LE d300985
feat(opt): :sparkles: add budget endpoints
3FE3LE 8fc9c8f
chore(global): :memo: generate openapi.json file on root project & ad…
3FE3LE 3060eaf
style(server): :art: format & remove useless code
3FE3LE 25ac017
fix(opt): :fire: remove deprecated postacode property from destinatio…
3FE3LE 2336c78
fix(opt): :bug: add @isEnum decorator to class-validation works properly
3FE3LE aa74456
feat(opt): :zap: add Trip entity, remove useGuard from findOne from s…
3FE3LE 9f8db75
chore(global): :card_file_box: migration apply name change on destini…
3FE3LE b71a52c
chore(opt): :truck: rename destiny resources
3FE3LE a45e8c9
chore(opt): :adhesive_bandage: rename imports, & referencias to destiny
3FE3LE 2250f8e
feat(global): :sparkles: implement token-user verification to detect …
3FE3LE d94e7e3
docs(global): :memo: update Api documentation with swagger decorators
3FE3LE 9cf5e03
feat(opt): :sparkles: add getCurrency endponit
3FE3LE 04f6788
refactor(opt): :poop: set actual param on controllers to query
3FE3LE 42121a4
fix(opt): :bug: fix parameter on controllers routes
3FE3LE 8180173
chore(global): :card_file_box: add MXN currency to db seed
3FE3LE 0de6b4c
fix(server): :bug: fix error handling on fail DB connection
3FE3LE 02a7d1a
feat(opt): :zap: add error handling on trip.service.ts
3FE3LE 08d747c
chore(global): :card_file_box: migration add placeId property on Dest…
3FE3LE a9fedc3
feat(opt): :sparkles: upateTrip service
3FE3LE 3e8944d
chore(opt): :card_file_box: migration, se new default value on trips
3FE3LE 1d5e9d4
feat(opt): :card_file_box: add expenses models & respective migrations
3FE3LE f31b316
perf(global): :arrow_up: upgrade prisma
3FE3LE 39637a9
fix(opt): :pencil2: fix trip property name from Expense to expenses
3FE3LE 51c65d0
feat(opt): :sparkles: now can add current user as member of created t…
3FE3LE 9ae2f5d
fix(docker): Update Node.js base image and fix build and run commands
3FE3LE c64c65f
fix(docker): Update Node.js base image to version 22.14.0-alpine3.20
3FE3LE bfc7b08
perf: :arrow_up: add migration SQL file to update primary keys and dr…
3FE3LE File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verify Node.js command syntax
The commands
node --run buildandCMD ["node", "--run", "start:prod"]use the--runflag which is not a standard Node.js flag. This appears to be an error and may prevent the container from building and running correctly.The correct commands should likely be:
Please verify the intended command syntax.
Also applies to: 26-26