fix: updated documentLoader to load stored JSON#27
Conversation
…and read from the stored json
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
📝 WalkthroughWalkthroughIntroduced bundled JSON-LD context files and modified the document loader to resolve hardcoded contexts locally instead of remote preloading. Added corresponding tests to verify context resolution and HTTP behavior for schemata and non-schemata URLs. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
🎉 This PR is included in version 6.10.3 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Summary
OA V3 is deprecated and the schemata is being used in V3. This PR is to update the context loader so that it will detect schemata.openattestation.com and load the stored context JSON instead of calling the endpoint.
Changes
Summary by CodeRabbit
Tests
New Features
Refactor