Skip to content

Feat: Add reports for stats from Trello (Issue #19) - #20

Merged
zoidy merged 20 commits into
mainfrom
19-feat-create-trello-report-for-curator-stats
Oct 2, 2025
Merged

Feat: Add reports for stats from Trello (Issue #19)#20
zoidy merged 20 commits into
mainfrom
19-feat-create-trello-report-for-curator-stats

Conversation

@zoidy

@zoidy zoidy commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator

Description
Uses the same scaffolding as the existing redata_reports to gather data from Trello to send to the LookerStudio Dashboard. The overall code was refactored to enable some reuse of secrets and general utility functions by leveraging DO's lib/ folder (refer to the DO build process documentation for how it works.

See #19

Documentation Update

  • I have updated README.md and other relevant documentation
  • No documentation update is needed

Implementation Notes
When running locally, the shared functions are loaded by adding lib/ to the path Python looks for modules in using sys.append. When deployed to DO, the build script copies the files in that folder to the same directory as the function. The sys.append line is still there but does nothing since the files will all be in the same folder.

@zoidy zoidy linked an issue Jul 18, 2025 that may be closed by this pull request
1 task
zoidy added 14 commits July 18, 2025 18:01
Only prints the available custom fields for each card for now
…code

The filter was excluding some cards that should not have been excluded
Debugging code can be turned off with the `debug` variable
The API returns a max of 1000 cards which requires pagination which is not implemented yet
Increase python version to 3.11 (highest available on DO without needing to install custom). Changes were needed to support it (mainly f-strings which were using 3.12 features)
@zoidy
zoidy marked this pull request as ready for review July 29, 2025 19:05
@zoidy
zoidy requested a review from HafeezOJ July 29, 2025 19:05

@HafeezOJ HafeezOJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved

@zoidy
zoidy merged commit aa0a35d into main Oct 2, 2025
1 check passed
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.

Feat: Create Trello report for curator stats

2 participants