Skip to content

Conversation

@Sveb
Copy link
Collaborator

@Sveb Sveb commented Oct 21, 2025

Component Created: DetailMedia.vue

Stories: ~/stories/DetailMedia.stories.js

Spec: ~/stories/DetailMedia.spec.js

Notes:
Screenshot 2025-10-21 at 13 34 10

Checklist:

  • I checked that it is working locally in the dev server
  • I checked that it is working locally in the storybook
  • I checked that it is working locally in the
    library-website-nuxt dev server
  • I added a screenshot of it working
  • UX has reviewed and approved this
  • I assigned this PR to someone on the dev team to review
  • I used a conventional commit message
  • I assigned myself to this PR

Copy link
Contributor

@pghorpade pghorpade left a comment

Choose a reason for hiding this comment

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

👍

@@ -0,0 +1,12 @@
describe('Funkhaus / DetailMedia', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

We dont need this file

@@ -0,0 +1,184 @@
import { computed } from 'vue'
Copy link
Contributor

Choose a reason for hiding this comment

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

we dont need this file

@@ -0,0 +1,140 @@
<script setup lang="ts">
Copy link
Contributor

Choose a reason for hiding this comment

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

Now that we have DLViewer migrated to Vue3 in this component library we dont need this file.

Copy link
Contributor

@pghorpade pghorpade left a comment

Choose a reason for hiding this comment

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

comment left on the other message

@@ -0,0 +1,115 @@
.detail-media,
Copy link
Contributor

Choose a reason for hiding this comment

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

@Sveb remove this file too if not used

Copy link
Contributor

@pghorpade pghorpade left a comment

Choose a reason for hiding this comment

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

👍

@github-actions github-actions bot temporarily deployed to storybook--pull_request_target-846 November 7, 2025 20:22 Inactive
Copy link
Contributor

@pghorpade pghorpade left a comment

Choose a reason for hiding this comment

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

👍

@pghorpade pghorpade merged commit 8891ff4 into UCLALibrary:main Nov 7, 2025
14 checks passed
@pghorpade
Copy link
Contributor

🎉 This PR is included in version @ucla-library-monorepo/ucla-library-website-components-v1.50.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants