Skip to content

Content Page refactor#175

Merged
rosemcc merged 44 commits into
masterfrom
content-pages-refactor
Nov 23, 2021
Merged

Content Page refactor#175
rosemcc merged 44 commits into
masterfrom
content-pages-refactor

Conversation

@Trombach
Copy link
Copy Markdown
Contributor

@Trombach Trombach commented Nov 15, 2021

In this pull request:

  • separated content list components from main content pages
  • breadcrumb component now loads parent subHubs based on current slug from route
  • refactor of async flow on content pages
  • default images for standard cards
  • Removed icons from list pages etc, and they will be eventually removed from the content model
  • Removed unused gql queries

commit 2f22e09
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:31:14 2021 +1300

    fixed single quotes

commit 2dc309e
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:12:24 2021 +1300

    removed subhub list from subhub component

commit 53b9847
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:11:23 2021 +1300

    removed software list from software component

commit fcb5263
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:09:07 2021 +1300

    removed service list from service component

commit 9207298
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:07:22 2021 +1300

    removed funding list from funding component

commit f5091c0
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:04:12 2021 +1300

    removed event list from event component

commit bd60389
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 15:00:24 2021 +1300

    removed all equipment query

commit feb33a4
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 14:53:26 2021 +1300

    removed equipment list from equipment component

commit 9c811a8
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 14:49:28 2021 +1300

    removed case study list from case study component

commit 22c4b6d
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 14:43:57 2021 +1300

    removed articles list from article component

commit 8c3e32f
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 14:19:33 2021 +1300

    broke out subhub list

commit b0bbef7
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 13:46:42 2021 +1300

    broke out software list

commit 0ce590b
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 13:33:38 2021 +1300

    broke out service list

commit ac7f5cd
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 13:04:01 2021 +1300

    changed import for funding route

commit 7b0289b
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 12:35:12 2021 +1300

    broke out funding list

commit 877c674
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 11:48:14 2021 +1300

    broke out event list

commit 8f14525
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 11:28:38 2021 +1300

    broke out equipment list

commit 4b76f8b
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 11:17:19 2021 +1300

    broke out case study list

commit 51d557c
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 11:06:45 2021 +1300

    unsubscribe on destroy

commit 8544616
Merge: c48690f 486d289
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 10:32:34 2021 +1300

    Merge branch 'master' into build-new-content-types-list-page

commit c48690f
Author: Luke <lukas.trombach@auckland.ac.nz>
Date:   Thu Oct 14 10:32:12 2021 +1300

    broke out article list
@Trombach Trombach requested a review from rosemcc November 15, 2021 21:36
@Trombach Trombach marked this pull request as ready for review November 16, 2021 02:46
Comment thread research-hub-web/src/app/components/casestudys/case-study/case-study.component.ts Outdated
@rosemcc
Copy link
Copy Markdown
Contributor

rosemcc commented Nov 19, 2021

Use page title service on the list pages

Copy link
Copy Markdown
Contributor

@rosemcc rosemcc left a comment

Choose a reason for hiding this comment

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

Hey Lukas, looks good to me. I've added a few minor changes that you might want to just check and make sure I didn't mess anything up ;)
Unit and e2e tests are passing.

@rosemcc rosemcc merged commit efa54c2 into master Nov 23, 2021
@rosemcc rosemcc deleted the content-pages-refactor branch November 23, 2021 21:48
@rosemcc rosemcc mentioned this pull request Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants