Skip to content

feat: implement mapping actions and enhance flow#21

Merged
MaximilianAnzinger merged 47 commits into
mainfrom
feature/ui-db-integration
Jan 12, 2026
Merged

feat: implement mapping actions and enhance flow#21
MaximilianAnzinger merged 47 commits into
mainfrom
feature/ui-db-integration

Conversation

@markstockhausen

@markstockhausen markstockhausen commented Dec 16, 2025

Copy link
Copy Markdown
Contributor

Description

Streamline the Session page UI by removing onboarding/dashboard links, adding competency badges with tooltips, and surfacing keyboard shortcut hints (Undo, Skip, Add Relation). Improve the layout with a full-height session section, reduced footer spacing, and static card styling (no hover animations). On the backend, load relationship types from the database and create/delete competency relationships when actions are completed or undone.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as
    expected)
  • 📝 Documentation update
  • 🎨 Style/formatting changes (no functional changes)
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvements
  • 🔧 Configuration changes
  • 🧪 Test additions or modifications
  • 🚀 Deployment/DevOps changes

Related Issues

Changes Made

  • Client: redirect onboarding; remove dashboard links; add competency badge tooltips; add gray Kbd shortcut pills (Undo/Skip/Add Relation) with tooltip delay/position controls; center relation selector with arrow and badges; make cards static (no hover animations); full-height session section with trimmed padding and reduced footer spacing.
  • Server: fetch relationship types from DB; create/delete competency relationships via actions on completion/undo.
  • Database: CRUD for competency relationships (create on completion, delete on undo) using existing Prisma models.
  • Other: added a reusable tooltip utility component.

Testing

Test Cases

  • Existing functionality still works
  • New functionality works as expected
  • Edge cases handled appropriately

Manual Testing

  • Tested in development environment
  • Tested with different user roles/permissions (if applicable)

Screenshots/Videos

sessionPreview

vtotalova and others added 30 commits November 25, 2025 12:50
- Add unique constraint on CompetencyRelationship to prevent duplicates
- Add database indexes on CompetencyResourceLink foreign keys for performance
- Make userId optional in CompetencyResourceLink with SetNull on delete
- Add error logging and visibility in migration system
- Move database docs to root as DATABASE.md
- Sync package-lock.json with package.json (TypeScript 5.9.3)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Co-authored-by: Maximilian Anzinger <44003963+MaximilianAnzinger@users.noreply.github.com>
Co-authored-by: Maximilian Anzinger <44003963+MaximilianAnzinger@users.noreply.github.com>
…om:ls1intum/memo into feature/setup-domain-core-db-communication
@markstockhausen

Copy link
Copy Markdown
Contributor Author

To be merged after #16 and #18

@MaximilianAnzinger

Copy link
Copy Markdown
Collaborator

@markstockhausen can you resolve the merge conflict?

@github-actions github-actions Bot removed database 🗃️ Database related changes docker 🐳 Docker/deployment changes labels Dec 23, 2025
@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@ls1intum ls1intum deleted a comment from github-actions Bot Dec 23, 2025
@markstockhausen

Copy link
Copy Markdown
Contributor Author

All merge conflicts resolved, ready to merge 👌

@vtotalova vtotalova left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Comment thread app/actions/competency_relationships.ts Outdated
@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

📊 Code Statistics

🚨 Potential Issues

✅ No console statements found
⚠️ Found 'any' or 'unknown' types - consider using more specific types

@markstockhausen

Copy link
Copy Markdown
Contributor Author

@MaximilianAnzinger Can you please merge this PR?

@MaximilianAnzinger
MaximilianAnzinger merged commit cb3a12d into main Jan 12, 2026
8 checks passed
@MaximilianAnzinger
MaximilianAnzinger deleted the feature/ui-db-integration branch January 12, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client-application 🎨 Client-side/UI changes configuration ⚙️ Configuration file changes dependencies 📦 Dependency updates documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: connect mapping UIs to Prisma/PostgreSQL

4 participants