Skip to content

Update to latest version #29

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
SergioNR merged 9 commits into from
Dec 26, 2025
Merged

Update to latest version #29

SergioNR merged 9 commits into from
Dec 26, 2025

Conversation

@SergioNR
Copy link
Member

@SergioNR SergioNR commented Dec 26, 2025

No description provided.

SergioNR and others added 9 commits December 11, 2025 23:20
@SergioNR
FIX TYPO analysisPresignedUploadUrl --> analysisEntryPresignedUploadUrl
ec55088
@SergioNR
MOVE transcript transformer into its own logical function
a546469
@SergioNR
Standardize time formatting backup functionality
c560dd1
@SergioNR
REMOVE references to analysisId & analysisStatus in /api/v1/analysisE…
7b7dc98 
…ntry

These will be handled in the backend
@SergioNR
Revert "REMOVE references to analysisId & analysisStatus in /api/v1/a…
394f3a2 
…nalysisEntry"

This reverts commit 7b7dc98.
@SergioNR
remove unneeded analysisEntryStatus attribute - will be handled in BE
5be9f73
@SergioNR
Reapply "REMOVE references to analysisId & analysisStatus in /api/v1/…
013799c 
…analysisEntry"

This reverts commit 394f3a2.
@SergioNR
Merge pull request #28 from UXcaptain/refactor-transcription-service-…
6161dc1 
…into-monolith

UPDATE to refactor backend transcription microservice into the main monolith
@SergioNR
UPDATE github actions to create dated image to be able to rollback
ae4ead3
@kiloconnect
Copy link

kiloconnect bot commented Dec 26, 2025

✅ No Issues Found

9 files reviewed | Confidence: 95% | Recommendation: Merge

Review Details

Files:

  • .github/workflows/docker:build&push.yaml - CI/CD improvements with Docker metadata
  • src/utils/transcriptTransformer.js - NEW: Centralized transcript transformation utility
  • src/components/partials/VideoPlayerSidebar.jsx - Transcript handling refactor
  • src/pages/user/VideoPlayerPage.jsx - Transcript transformation integration
  • src/components/partials/AnalysisStepNavigator.jsx - Parameter renaming & API simplification
  • src/contexts/MediaPermissionsContext.jsx - Parameter updates
  • src/pages/participate/ParticipateStepRouter.jsx - Parameter renaming
  • src/pages/participate/ParticipateWrapper.jsx - Parameter renaming
  • src/pages/participate/steps/RecordingStep.jsx - Parameter renaming

Changes Analyzed:

  • Security: No security issues found
  • Functionality: Parameter renaming maintains API compatibility, transcript transformation improves consistency
  • Performance: Added memoization for transcript transformation, removed unnecessary API parameters
  • Code Quality: Good architectural improvements with centralized utility functions

Key Improvements:

  1. Docker workflow: Added metadata action for better CI/CD tagging
  2. Transcript handling: Centralized transformation logic in dedicated utility
  3. API optimization: Simplified unnecessary parameters from API calls
  4. Consistency: Standardized parameter naming across components

Issues Found: 1 minor suggestion (environment variable case consistency)

@SergioNR SergioNR merged commit 11f0904 into latest Dec 26, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SergioNR