Closed amindadgar closed 3 months ago
The recent changes across several files in the GitHub integration involve transitioning from using organization IDs to organization names for identifying and fetching organization details. This update affects the get_github_configs
function, the extract_github_organization
task, and related test cases. Additionally, the get_learning_platforms
method now includes organization IDs in its return data structure, and test cases have been updated to reflect these changes.
File Path | Change Summary |
---|---|
dags/github.py |
Switched from using organization IDs to organization names in the get_github_configs function and extract_github_organization task. |
dags/hivemind_etl_helpers/src/utils/modules/github.py |
Updated get_learning_platforms to include organization_id in the returned data and changed metadata_name parameter to "account". |
dags/hivemind_etl_helpers/tests/integration/test_github_get_communities_org.py |
Updated test cases to include organization names in the account field of the metadata dictionary. |
sequenceDiagram
participant User
participant GitHubIntegration
participant GitHubAPI
User->>GitHubIntegration: Request organization data
GitHubIntegration->>GitHubAPI: Fetch data using organization names
GitHubAPI-->>GitHubIntegration: Return organization data
GitHubIntegration-->>User: Provide organization data
get_all_organization
task to fetch organization modules.In the world of code so bright and clear,
Organizations by name now appear,
Fetching details with a graceful cheer,
IDs replaced, no longer near.
Tests updated, all in gear,
GitHub flows, no need to fear,
For data's path is now sincere.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
New Features
Tests