Skip to content

Moss/trim audio feature#777

Open
Lomacar wants to merge 134 commits intodevfrom
Moss/trim-audio-feature
Open

Moss/trim audio feature#777
Lomacar wants to merge 134 commits intodevfrom
Moss/trim-audio-feature

Conversation

@Lomacar
Copy link
Copy Markdown
Collaborator

@Lomacar Lomacar commented Mar 1, 2026

This PR includes work from the following branches: AssetAudio, Moss/unmerging, Jesus/expo-54-2026, rafael/new-recording-view-b.

Main changes on this branch are to RecordingView and BibleAssetsView. Added trimming modal, accessible from the selection control panel. The selection control panel was also reconfigured and unified between the two views, could be further improved perhaps. There was a bunch of refactoring of functionality between the two views while I was at it too, so they now share logic in some hooks and stuff rather than having separate implementations.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
langquest Ready Ready Preview, Comment Mar 10, 2026 8:17pm

Request Review

@supabase
Copy link
Copy Markdown

supabase bot commented Mar 1, 2026

Updates to Preview Branch (Moss/trim-audio-feature) ↗︎

Deployments Status Updated
Database Tue, 10 Mar 2026 20:16:02 UTC
Services Tue, 10 Mar 2026 20:16:02 UTC
APIs Tue, 10 Mar 2026 20:16:02 UTC

Tasks are run on every commit but only new migration files are pushed.
Close and reopen this PR if you want to apply changes from existing seed or migration files.

Tasks Status Updated
Configurations Tue, 10 Mar 2026 20:16:10 UTC
Migrations Tue, 10 Mar 2026 20:16:13 UTC
Seeding Tue, 10 Mar 2026 20:16:13 UTC
Edge Functions Tue, 10 Mar 2026 20:16:21 UTC

View logs for this Workflow Run ↗︎.
Learn more about Supabase for Git ↗︎.

@socket-security
Copy link
Copy Markdown

socket-security bot commented Mar 1, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedeslint-plugin-react@​7.37.59710010083100
Addedeslint-plugin-react-compiler@​19.1.0-rc.21001009290100
Addedeslint-plugin-react-hooks@​7.0.110010010096100

View full report

Copy link
Copy Markdown

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Native Build Required - See the comment below for details.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 1, 2026

⚠️ Native Build Required - Manual Approval Needed

This PR requires a native build because the fingerprint has changed compared to the base branch.

📋 Fingerprint diff
- Changed (before): Path: eas.json | Hash: 49bcbd213218f3c73564a9d08ac7c96df2b7eeb9 | Reasons: easBuild
+ Changed (after): Path: eas.json | Hash: f693618023380fa44ee50c424e57cf39ac7881c9 | Reasons: easBuild

- Changed (before): Path: modules/microphone-energy/android | Hash: f5c225ac74b256bd9dff5756df885a0ce064a5ee | Reasons: expoAutolinkingAndroid
+ Changed (after): Path: modules/microphone-energy/android | Hash: 9ba1dd5c3b23712e1c27422ac2e079bec7f76998 | Reasons: expoAutolinkingAndroid

- Changed (before): Path: modules/microphone-energy/ios | Hash: 0a0e6e1241117843e94fa529dedf69c2bce7b5c9 | Reasons: expoAutolinkingIos
+ Changed (after): Path: modules/microphone-energy/ios | Hash: 591bfee0e17dcf69fccff005f2a15ded48f5dfa5 | Reasons: expoAutolinkingIos
🔍 Full JSON diff (for debugging)
[
  {
    "op": "changed",
    "beforeSource": {
      "type": "file",
      "filePath": "eas.json",
      "reasons": [
        "easBuild"
      ],
      "hash": "49bcbd213218f3c73564a9d08ac7c96df2b7eeb9"
    },
    "afterSource": {
      "type": "file",
      "filePath": "eas.json",
      "reasons": [
        "easBuild"
      ],
      "hash": "f693618023380fa44ee50c424e57cf39ac7881c9"
    }
  },
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "modules/microphone-energy/android",
      "reasons": [
        "expoAutolinkingAndroid"
      ],
      "hash": "f5c225ac74b256bd9dff5756df885a0ce064a5ee"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "modules/microphone-energy/android",
      "reasons": [
        "expoAutolinkingAndroid"
      ],
      "hash": "9ba1dd5c3b23712e1c27422ac2e079bec7f76998"
    }
  },
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "modules/microphone-energy/ios",
      "reasons": [
        "expoAutolinkingIos"
      ],
      "hash": "0a0e6e1241117843e94fa529dedf69c2bce7b5c9"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "modules/microphone-energy/ios",
      "reasons": [
        "expoAutolinkingIos"
      ],
      "hash": "591bfee0e17dcf69fccff005f2a15ded48f5dfa5"
    }
  }
]

Action Required: After confirming with the team, another team member must approve this PR to proceed with merging.

👉 Approve this PR - Click "Review changes/Submit review" → "Approve" to dismiss this review and allow merging.

- I don't understand why I ALWAYS have to do this even when I have configured Cursor and the repo to do things correctly in the first place
@cursor
Copy link
Copy Markdown

cursor bot commented Mar 10, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 20.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants