Skip to content

fix: expose (non-hso) french fields to non-hso users#452

Merged
AlexCLeduc merged 1 commit intomainfrom
expose-french-fields-to-normal-users
Jan 22, 2026
Merged

fix: expose (non-hso) french fields to non-hso users#452
AlexCLeduc merged 1 commit intomainfrom
expose-french-fields-to-normal-users

Conversation

@AlexCLeduc
Copy link
Contributor

not a bug fix, just something the business asked for

def get_success_url(self):
messages.success(self.request, tm("saved_successfully"))
return reverse("view_indicator", kwargs={"pk": self.object.pk})
return reverse("edit_indicator", kwargs={"pk": self.object.pk})
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also quietly changed this redirect, I found it pretty annoying (hopefully the users agree with me...)

@AlexCLeduc AlexCLeduc merged commit d3126ec into main Jan 22, 2026
2 checks passed
@AlexCLeduc AlexCLeduc deleted the expose-french-fields-to-normal-users branch January 22, 2026 17:34
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