Skip to content

Conversation

@briannval
Copy link
Contributor

@briannval briannval commented Oct 16, 2025

POC for Kickstart Investment Service

DO NOT REVIEW THIS, review the other Kickstart PR instead.

Closes:
#564
#565
#566

@briannval briannval force-pushed the kickstart-investment-service branch from 861c36d to 65aa945 Compare October 17, 2025 01:30
@briannval briannval force-pushed the kickstart-investment-service branch from 626d6b2 to 2d64d3a Compare October 17, 2025 03:39
@briannval briannval force-pushed the kickstart-investment-service branch from 9d8daf3 to 579dd79 Compare October 17, 2025 03:43
@briannval briannval force-pushed the kickstart-investment-service branch from c08f9b9 to 6a139c6 Compare October 17, 2025 03:48
path: invest
method: post
cors: true
authorizer: aws_iam No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check other services to see how to access shared authorizer

Resource:
- "arn:aws:dynamodb:us-west-2:432714361962:table/biztechRegistrations${self:provider.environment.ENVIRONMENT}"
- "arn:aws:dynamodb:us-west-2:432714361962:table/biztechTeams${self:provider.environment.ENVIRONMENT}"
- "arn:aws:dynamodb:us-west-2:432714361962:table/biztechInvestments${self:provider.environment.ENVIRONMENT}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

may need transact writes (with denormalized table, since we're going to be writing to multiple users in a team at once)

export const userStatus = async (event, ctx, callback) => {
/*
Responsible for:
- Fetching user current balance
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's confirm with gautham before we go ahead with this

@briannval briannval mentioned this pull request Nov 2, 2025
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.

3 participants