Skip to content

Initializes DynamoDB persistence layer into sync flow#12

Open
tamaradement wants to merge 1 commit into
mainfrom
tamara/init-dynamodb-integration
Open

Initializes DynamoDB persistence layer into sync flow#12
tamaradement wants to merge 1 commit into
mainfrom
tamara/init-dynamodb-integration

Conversation

@tamaradement

Copy link
Copy Markdown
Collaborator

PR Description

  • Adds a new DYNAMODB_TABLE_NAME environment variable.
  • Begins integrating the DynamoDB persistence layer into the main sync flow.
    • For each filtered Clockify request, checks whether the request already exists in DynamoDB and logs the result.

Steps I Took to Verify

  • Ran the sync script against a newly created Clockify request and verified the request was reported as not existing in DynamoDB.
  • Inserted a corresponding item into DynamoDB and ran the same script again. Verified the request was reported as already existing in DynamoDB.

@tamaradement tamaradement marked this pull request as draft June 30, 2026 20:55
@tamaradement tamaradement self-assigned this Jul 4, 2026
@tamaradement tamaradement requested a review from dacut July 4, 2026 19:06
@tamaradement tamaradement marked this pull request as ready for review July 4, 2026 19:07
tableName,
)

for _, req := range env.Requests {

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

My thinking here is that this is just the first step in integrating the persistence layer. The next change will use this to filter out requests that have already been processed before creating gCal events. Once that's in place, I'll probably move this logic into core so main.go stays orchestration-focused.

@dacut dacut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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.

2 participants