Skip to content

Conversation

@fago
Copy link
Contributor

@fago fago commented Mar 10, 2021

No description provided.

@codecov
Copy link

codecov bot commented Mar 16, 2021

Codecov Report

Merging #14 (6217d00) into 1.x (0b813ff) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              1.x      #14   +/-   ##
=======================================
  Coverage   96.00%   96.00%           
=======================================
  Files           1        1           
  Lines          25       25           
  Branches        8        8           
=======================================
  Hits           24       24           
  Misses          1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0b813ff...6217d00. Read the comment docs.

@fago fago force-pushed the feature/fix-hydration branch from 1ff9379 to bf94abb Compare March 17, 2021 17:17
@fago fago force-pushed the feature/fix-hydration branch from 8b0816f to 471707b Compare March 18, 2021 13:51
@fago fago requested a review from maximilianmikus March 18, 2021 14:57
/**
* Returns an object for rendering as Vue component.
*/
this.$currentPage.contentComponent = () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this one main reason for the warnings (it'S not able to serialize this)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

was unused anyway

// Request made and server responded. If we get a proper Drupal error
// page, apply it and propagate the error code.
if (error.response.data && error.response.data.title && error.response.data.content) {
this.processPageResponse(error.response.data, error.response.status)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this takes over drupal error pages if possibles - nothing special todo then to make them apply.
if necessary statusCode property is there in case the frontend wants to differentiate. It won't trigger nuxt error handling and default nuxt error pages. So we can have two level of error handling:

  1. regular 403/404 pages, cms controlled
  2. nuxt errors, solely frontend controlled.

@fago fago merged commit 8c770ff into 1.x Mar 18, 2021
@fago fago deleted the feature/fix-hydration branch March 18, 2021 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants