Skip to content

feat: import advanced-search module#236

Merged
at88mph merged 2 commits intoopencadc:mainfrom
WenbinWL:feature/advanced-search-migration
Oct 16, 2025
Merged

feat: import advanced-search module#236
at88mph merged 2 commits intoopencadc:mainfrom
WenbinWL:feature/advanced-search-migration

Conversation

@WenbinWL
Copy link

@WenbinWL WenbinWL commented Oct 16, 2025

Summary

  1. Import the full Advanced Search codebase into advanced-search/, preserving integration/unit tests, docs, WAR build, and Docker assets while dropping its standalone Gradle wrapper.
  2. Wire the module into the multi-project build (settings.gradle), aligning it with the existing caom2ui toolchain.
  3. Update shared components to compile on Java 11: switch TAPSearcher to tapJob.owner and declare Servlet API dependencies as compileOnly/testImplementation so the container supplies them.
  4. Run ./gradlew clean build -x intTestChrome -x intTestFirefox successfully across the repo; Selenium tasks remain defined but still skipped pending environment details.
  5. Added an Advanced Search section to README.md.

@WenbinWL WenbinWL force-pushed the feature/advanced-search-migration branch from 31cedf9 to a82451b Compare October 16, 2025 00:35
@at88mph at88mph merged commit 98ef07d into opencadc:main Oct 16, 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.

2 participants