Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Modified the RequestResetView post error redirect to work with base_p…#7006

Merged
wardi merged 3 commits into
ckan:masterfrom
JVickery-TBS:fix/request-reset-error-redirect
Mar 31, 2023
Merged

Modified the RequestResetView post error redirect to work with base_p…#7006
wardi merged 3 commits into
ckan:masterfrom
JVickery-TBS:fix/request-reset-error-redirect

Conversation

@JVickery-TBS
Copy link
Copy Markdown
Contributor

…ath and locale.

Fixes #

Modified the redirect during the password reset form error. Changed it so the redirect works with base_path and locale.

Features:

  • includes tests covering changes
  • includes updated documentation
  • includes user-visible changes
  • includes API changes
  • includes bugfix for possible backport

Please [X] all the boxes above that apply

@JVickery-TBS
Copy link
Copy Markdown
Contributor Author

@wardi I closed #6999 and create this one as I needed to hard reset my ckan fork. This PR should now only have the two commits, one for the code change and one for the change log file.

@wardi wardi self-assigned this Aug 11, 2022
@wardi
Copy link
Copy Markdown
Contributor

wardi commented Sep 21, 2022

@JVickery-TBS there's a small lint error to fix on this (the other error looks like it might be unrelated)

@JVickery-TBS
Copy link
Copy Markdown
Contributor Author

@wardi the lint test is not from my changes, so should I still fix the syntax?

@JVickery-TBS
Copy link
Copy Markdown
Contributor Author

@wardi All tests are passing now. :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants