Skip to content

fix(webapp): MCA-201 Resolve getServerSideProps Missing Return Object Error#1091

Draft
willgibson-madetech wants to merge 4 commits intomainfrom
MCA-201-getServerSideProps-error
Draft

fix(webapp): MCA-201 Resolve getServerSideProps Missing Return Object Error#1091
willgibson-madetech wants to merge 4 commits intomainfrom
MCA-201-getServerSideProps-error

Conversation

@willgibson-madetech
Copy link
Copy Markdown
Contributor

Context

Aim to resolve https://madetech-team-b91dpvs1.atlassian.net/browse/MCA-201 and reduce log noise.

Changes in this pull request

  • Add failing test
  • Return empty, but valid object when no rules match
  • Adjust given a non-existent currentUseId test, which was already not asserting what it said on the tin

Guidance to review

This will need human testing to be sure we haven't changed the desired behaviour in some way that is not covered by automated tests.

Things to check

  • Human testing has taken place to be sure we haven't changed the desired behaviour in some way that is not covered by automated tests.

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.

1 participant