Merged
Conversation
- Implement comprehensive MongoDB brand color system with CSS custom properties - Add ExpandableTable component with modern pill-shaped Show More/Show Less button - Update all component styling to use MongoDB brand colors (Forest, Evergreen, Mint, Slate) - Replace gradient styling with solid MongoDB brand colors throughout - Change background from Mint to White for cleaner appearance - Update comment boxes with MongoDB branding (Mint background, Forest border) - Redesign pagination to text-link style with animated underlines - Update search modal buttons (Clear, Close, Search Movies) from blue to Forest green - Enhance hover effects, transitions, and accessibility across all components - Improve responsive design and focus states for better UX
CVE fix per Dependabot alert #22
fix: bump python-multipart>=0.0.22 for security fix
CVE fix per Dependabot alert #23
fix: bump next to 16.1.5 for security fix
- Remove hardcoded year bounds (1800-2030) from all three backends - Backend aggregations now only validate year field is numeric (: number) - Add fetchYearBounds() on client to dynamically detect min/max years from data - Move dataset warning above Year field label in FilterBar - Update tests to reflect removed year bound validation Backends modified: Java Spring, Express, Python FastAPI Client modified: FilterBar.tsx, api.ts
The Atlas CLI local deployment creates a replica set (myLocalRs1) that advertises hostname 'mylocalrs1'. Without directConnection=true, the MongoDB driver attempts server discovery and fails to resolve this hostname in CI, causing all tests to fail with: MongoServerSelectionError: getaddrinfo EAI_AGAIN mylocalrs1
- Quote all references to $GITHUB_STEP_SUMMARY to prevent bash syntax errors when the variable is empty or unset - Add guard at script start to gracefully skip summary generation if GITHUB_STEP_SUMMARY is not set This fixes the 'Generate Test Summary' step failure in CI where unquoted variable references caused 'echo >> ' syntax errors with set -e enabled.
Create index on comments.movie_id after data restore to fix the /reportingByComments aggregation timeout. Without this index, the $lookup operation performs a full collection scan (41K comments x 21K movies) causing 5+ minute response times in CI vs ~700ms locally.
The previous commit added an index creation step that requires mongosh, but mongosh is not installed on the Ubuntu runner by default. This adds a step to download and install mongosh-2.3.8-linux-x64.
Co-authored-by: Dachary <dachary.carey@mongodb.com>
DOCSP-57250: Reflect dataset limitation around movie year (no data 2016+)
- #1 Movie Cards: Make entire card clickable, enforce consistent heights, tone down checkbox - #2 Top Toolbar: Remove batch buttons, add contextual bottom selection bar - #3 Filters Bar: Replace mint/green with neutral gray borders and backgrounds - #4 Navbar: Remove full-width green border and animated underline effect - #5 Aggregations: Use light gray for row hover, tone down comment pills and show more button - Additional: Remove bright green border from aggregations section headers All changes improve visual hierarchy and reduce competing visual elements per reviewer feedback on PR #75.
- Remove checkbox UI element from MovieCard component - Remove checkbox-related CSS styles - Update click handler to remove checkbox-specific logic - Preserve card's clickable selection behavior - Maintain visual selection state (border, background, shadow)
- Increase min-width from 550px to 650px - Increase max-width from 600px to 700px - Prevents Edit/Delete Selected buttons from stacking vertically - Maintains single-line layout when selecting 2+ movies
…f green - Replaced green border colors with neutral gray to match FilterBar styling - Removed mint green background on hover - Updated focus state to use gray outline instead of green - Ensures dropdown doesn't look selected when it's not
- Updated page.module.css to use Forest green and Evergreen for link states - Updated not-found.module.css to match MongoDB brand colors - Changed focus outline to use MongoDB Forest green - Ensures consistent branding across all navigation links
- Normal state: White background with green border (original style) - Hover state: Dark Forest green background with white text - Matches the inverted color pattern of Delete Selected button
…ondary - Changed Search Movies button to secondary style (white background with Forest green border) - Kept Add Movie button as primary action (solid Forest green background) - Updated Search Movies hover state to invert colors (dark green background with white text) - Creates clear visual hierarchy between primary and secondary actions
- Python FastAPI: Update fastapi, pydantic, voyageai, pytest, and other dependencies - Java Spring: Update spring-dotenv (5.1.0), commons-lang3 (3.20.0), langchain4j (1.11.0-beta19), impsort-maven-plugin (1.13.0) - Java Spring: Fix spring-dotenv artifact to springboot3-dotenv for Spring Boot 3 compatibility - Next.js Frontend: Update next (16.1.6), react (19.2.4), react-dom (19.2.4), eslint (9.39.2) All tests pass: - Python: 75 tests - Java: 92 tests - Frontend: Build successful
DOCSP-57631: Fix js-express GitHub workflow
DOCSP-55536: Apply MongoDB branding to mFlix application
Update dependencies across all backends and frontend
krollins-mdb
approved these changes
Feb 9, 2026
Collaborator
krollins-mdb
left a comment
krollins-mdb
left a comment
There was a problem hiding this comment.
LGTM. Thank you, @cbullinger!
Collaborator
Author
|
All tests pass and all three backends build successfully |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge the following
developmentfeatures intomainbranch: