feat: add forum(discourse) read endpoint view configs and smoke test scenarios#14
Conversation
Welcome To opensourceways CommunityHey @TommyLike , thanks for your contribution to the community. Bot Usage ManualI'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands. |
CLA Signature PassTommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
Linking Issue Notice@TommyLike , the pull request must be linked to at least one issue. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request migrates the module path from github.com/cncf/cora to github.com/opensourceways/cora across all Go files. It also introduces numerous new view configurations for the forum service, updates the field extraction logic to support dot-separated nested paths, and adds a comprehensive suite of integration test scenarios under scenarios/forum/. There are no review comments, and I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
…scenarios - Add view configs for 27 forum read endpoints (badges, categories, groups, posts, tags, topics, users) in internal/view/builtin.go - Support dot-notation RootField paths (e.g. "topic_list.topics") in internal/view/extract.go DetectItems - Create 27 new smoke scenario YAML files for forum read endpoints - Update existing forum scenarios (enable skip:false, add RootField) - Add smoke config file for forum service - Fix module import path to opensourceways/cora 26/30 forum smoke tests pass, 4 skipped (spec/test data limitations). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
TommyLike
force-pushed
the
feat/support-release
branch
from
a504ab0 to
fb0ea54
Compare
CLA Signature PassTommyLike, thanks for your pull request. All authors of the commits have signed the CLA. 👍 |
2 participants
Summary
Add view configs and smoke test scenarios for Discourse/Forum read endpoints.
Changes
View configs (
internal/view/builtin.go):RootFieldwith dot-notation (e.g.topic_list.topics) for nested response structuresRootField enhancement (
internal/view/extract.go):DetectItemsnow supports dot-separatedRootFieldpaths viaExtractField(), enabling multi-level nesting liketopic_list.topicsSmoke scenarios (
scenarios/forum/):Smoke test results: 26 passed, 0 failed, 4 skipped
Skipped scenarios (4)
topics-get-posttopics-get-external-idusers-get-by-externalusers-get-by-external-by-external🤖 Generated with Claude Code