Closed amindadgar closed 3 months ago
The recent changes improve the get_learning_platforms
function in the mediawiki.py
module by concatenating the base_url
with a new path
from platform metadata. Corresponding updates in the integration tests ensure the metadata
dictionary now includes a path
key, reflecting the new structure.
Files | Change Summary |
---|---|
dags/hivemind_etl_helpers/src/utils/modules/mediawiki.py |
Updated get_learning_platforms to concatenate base_url with path from platform metadata. |
dags/hivemind_etl_helpers/tests/integration/test_mediawiki_modules.py |
Modified metadata dictionary in tests to include a path key and split the URL into baseURL and path . |
sequenceDiagram
participant User
participant get_learning_platforms
participant PlatformMetadata
User->>get_learning_platforms: Call function
get_learning_platforms->>PlatformMetadata: Retrieve base_url and path
PlatformMetadata-->>get_learning_platforms: Return base_url and path
get_learning_platforms->>get_learning_platforms: Concatenate base_url and path
get_learning_platforms-->>User: Return updated URL
In code, we weave our tale so tight,
URLs split, paths now in sight,
Tests align with metadata's call,
Ensuring platforms stand tall.
A rabbit's touch, so light and swift,
Enhancing functions, a coder's gift.
🐇✨
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?
Updated to support both baseURL and path.
Summary by CodeRabbit
New Features
Tests