Skip to content
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.

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

refactor: APPS-2411 Add home page #760

Merged
merged 29 commits into from
Feb 20, 2024
Merged

refactor: APPS-2411 Add home page #760

merged 29 commits into from
Feb 20, 2024

Conversation

pghorpade
Copy link
Contributor

@pghorpade pghorpade commented Jan 25, 2024

Connected to APPS-2411

Page/Pages Created/updated:
pages/index.vue
pages/about/reports/index.vue
pages/impact/[year]/[slug].vue
pages/impact/[year]/index.vue
pages/impact/all.vue


Documented here:
Nuxt3 setup commit walk thru 1: https://youtu.be/jgpZEDLEt68?si=Extq7FbD9wiKf8OQ
Nuxt3 setup commit walk thru 2: https://youtu.be/McZd2vCmSAw?si=7bemksB_2yTjw-5i


Notes:
Add flexibleblocks to these pages once added to the vue3 component library

Elastic search is not integrated yet with plugins.
Replaced Apollo GraphQL client with nuxt-graphql-request, this is what we used in nuxt2 and is now nuxt3 compatible.
Using nuxt3 nitro server plugin to fetch global craft data for header and footer data.
Will move global notification alert data to default.vue later so it works on the client side and we will not have to regenerate the static site when editors add alerts to craft
Replaced antfu-eslint to nuxt-eslint-config and its typescript plugin for no-undef errors whenusing nuxt magic autoimports.
Will add typescript in later.

Checklist:

  • I added github label for semantic versioning
  • I double checked it looks like the designs
  • I completed any required mobile breakpoint styling
  • I completed any required hover state styling
  • I included a working spec file
  • I added notes above about how long it took to build this component
  • UX has reviewed this PR
  • I assigned this PR to someone to review

Copy link
Contributor

Percy Screenshots

In order to conserve our percy screenshot allowance, percy is not configured to run automatically. Please make sure the PR is ready and all other checks are passing, then start it manually: 1. Visit https://github.com/UCLALibrary/library-website-nuxt/actions/workflows/percy.yml 2. Click the 'Run workflow' button in the blue bar. 3. Select the correct branch for this PR and click 'Run workflow' again to confirm.

@pghorpade pghorpade changed the title fix: APPS-2411 Add home page refactor: APPS-2411 Add home page Jan 25, 2024
Copy link
Contributor

github-actions bot commented Jan 25, 2024

@github-actions github-actions bot temporarily deployed to pull_request-760 February 17, 2024 04:36 Inactive
@github-actions github-actions bot temporarily deployed to pull_request-760 February 17, 2024 04:47 Inactive
@github-actions github-actions bot temporarily deployed to pull_request-760 February 17, 2024 04:55 Inactive
@github-actions github-actions bot temporarily deployed to pull_request-760 February 17, 2024 07:44 Inactive
@github-actions github-actions bot temporarily deployed to pull_request-760 February 17, 2024 07:46 Inactive
…e prerender as listing pages are not complete, also finished moving library alert on client side in client plugin
@github-actions github-actions bot temporarily deployed to pull_request-760 February 20, 2024 22:11 Inactive
@pghorpade pghorpade merged commit e76bcc9 into nuxt3.x Feb 20, 2024
6 checks passed
@pghorpade pghorpade deleted the APPS-2411 branch February 20, 2024 23:40
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.

1 participant