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

fix: content page not found when reloading a page with an authenticated user #1198

Merged
merged 1 commit into from
Jun 29, 2022

Conversation

Eisie96
Copy link
Contributor

@Eisie96 Eisie96 commented Jun 29, 2022

PR Type

[ x ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

When an authenticated user reloads a content page, then the "Page Not Found" component is rendered. The reason for this is, that the needed personalization call is dispatched after the content page call, which should use the pgid.

Issue Number: Closes #

What Is the New Behavior?

The content page call is dispatched only after the personalization status is determined.

Does this PR Introduce a Breaking Change?

[ ] Yes
[ x ] No

Other Information

AB#77774

@Eisie96 Eisie96 requested a review from SGrueber June 29, 2022 11:56
@Eisie96 Eisie96 changed the title fix: content page not found when reloading a page with an unauthenticated user fix: content page not found when reloading a page with an authenticated user Jun 29, 2022
@Eisie96 Eisie96 merged commit 1e7af60 into develop Jun 29, 2022
@Eisie96 Eisie96 deleted the fix/reload-content-page branch June 29, 2022 13:16
shauke pushed a commit that referenced this pull request Jul 13, 2022
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