Skip to content

Conversation

@Alexgars73
Copy link
Contributor

Error in response in first load:
image

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 27, 2025
@Alexgars73
Copy link
Contributor Author

Alexgars73 commented Jul 28, 2025

@pedrobaeza could you remove the stale tag please?

@pedrobaeza pedrobaeza removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Aug 1, 2025
@pedrobaeza pedrobaeza added this to the 16.0 milestone Aug 1, 2025
@pedrobaeza
Copy link
Member

OK, but please enroll people to review it.

@Alexgars73
Copy link
Contributor Author

Alexgars73 commented Aug 1, 2025

@yvaucher @NICO-SOLUTIONS could you review please?

Copy link
Member

@NICO-SOLUTIONS NICO-SOLUTIONS left a comment

Choose a reason for hiding this comment

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

Havent seen this effect up to now but it makes sense!
in generel i would place the check for Content-Type == "application/json" before assigning the redirect_path. but thats academic.

LGTM!

@Alexgars73 Alexgars73 force-pushed the 16.0-fix-website_require_login branch 2 times, most recently from c5c4a3e to 58037bf Compare August 6, 2025 10:14
)

def test_dispatch_json_no_redirect(self):
"""Con Content-Type application/json no debe haber redirección."""
Copy link
Member

Choose a reason for hiding this comment

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

Please use English

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@tarteo done!

@Alexgars73 Alexgars73 changed the title [FIX] website_require_login: Error response [16.0][FIX] website_require_login: Error response Nov 13, 2025
@Alexgars73 Alexgars73 force-pushed the 16.0-fix-website_require_login branch from 58037bf to fb35027 Compare November 18, 2025 12:50
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.

4 participants