UCLALibrary / ucla-library-website-components

This is a library of Vue components that will be used in UCLA Library Nuxt websites.
Other
4 stars 1 forks source link

feat: APPS-2633 | APPS-2634 | APPS-2635 | APPS-2636 | APPS-2581 | APPS-2582 | APPS-2583 Convert BlockStaffList, BlockStaffArticle, SectionStaffList, SectionStaffSubjectLibrarian, BlockStaffDetail to vue3 & Fix BlockStaffSubjectLibrarian #483

Closed jendiamond closed 2 months ago

jendiamond commented 3 months ago

Connected to APPS-2633 | APPS-2634 | APPS-2635 | APPS-2636 | APPS-2581 | APPS-2582 | APPS-2583

Component Created:

Component Fixed:

Stories:

Spec:

Notes:

This PR includes code in Flexible/BannerFeatured.vue because the
Run CI Suite / vite (pull_request) was failing with this error:

src/lib-components/Flexible/BannerFeatured.vue(83,20): 
error TS2345: 
Argument of type '{ title: string; to: string; }'
is not assignable to parameter of type 'LocationItemType'.

Type '{ title: string; to: string; }' is missing the following properties from type 'LocationItemType': id, image, locationType, uri, and 7 more.
 ELIFECYCLE  Command failed with exit code 2.

I commented out the code:

  // Currently BlockCardWithImageFragment.gql comments out
  // line 96. # location {id,title}
  // if (block.content && block.content[0].location)
  // locations.push(block.content[0].location)

because as noted in the code BlockCardWithImageFragment.gql comments out line 96. # location {id,title} in the library-website-nuxt site

        ... on cardWithImage_externalContent_BlockType {
            id
            typeHandle
            title: titleGeneral
            image {
                ...Image
            }
            byline1
            byline2
            category
            # location {
            #     id
            #     title
            # }
            text: summary
            to: externalLink
        }

Checklist:

Screenshot 2024-04-03 at 4 11 09 PM

Screenshot 2024-04-04 at 11 25 40 AM

Screenshot 2024-04-05 at 9 58 14 AM

Nuxt Screenshot 2024-04-04 at 12 40 55 PM (2) Screenshot 2024-04-04 at 12 49 14 PM (2)

github-actions[bot] commented 2 months ago

🚀 Deployed on https://deploy-preview-483--ucla-library-storybook-vue3x.netlify.app

farosFreed commented 2 months ago

should we close this PR at this point? It seems most of the work has been moved elsewhere

jendiamond commented 2 months ago

Broke this into 5 smaller PRs