Skip to content

[18.0][IMP] Make is_redirect_home field always visible in user form - #3385

Open
andreampiovesana wants to merge 1 commit into
OCA:18.0from
andreampiovesana:patch-6
Open

[18.0][IMP] Make is_redirect_home field always visible in user form#3385
andreampiovesana wants to merge 1 commit into
OCA:18.0from
andreampiovesana:patch-6

Conversation

@andreampiovesana

@andreampiovesana andreampiovesana commented Dec 9, 2025

Copy link
Copy Markdown

set menu as initial view always activable
'because it's better for UX

set menu as initial view always activable
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @SplashS, @Tardo,
some modules you are maintaining are being modified, check this out!

@ivs-cetmix

Copy link
Copy Markdown
Member

@andreampiovesana thank you for your contribution!
Could you please update the PR name and put the correct tag ([IMP]) in it?
Also please update the commit message in accordance with the OCA guidelines. Eg

[IMP] web_seponsive user settings

Put here the reason behind the changes. 
Eg 'because it's better for UX

@andreampiovesana andreampiovesana changed the title [18.0][ADD] Make is_redirect_home field visible in user form [18.0][IMP] Make is_redirect_home field always visible in user form Apr 8, 2026
@andreampiovesana

Copy link
Copy Markdown
Author

please merge

@ivs-cetmix ivs-cetmix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code review LGTM

@pedrobaeza

Copy link
Copy Markdown
Member

@andreampiovesana

andreampiovesana commented Apr 11, 2026

Copy link
Copy Markdown
Author

is an ultra simple change... we can close thi PR

@pedrobaeza pedrobaeza added this to the 18.0 milestone Apr 11, 2026
@pedrobaeza

Copy link
Copy Markdown
Member

As you prefer. Anyway, I think the invisible condition is because if you have set an initial action, there's no redirection to home, so I think it's correct as it is now, or am I wrong?

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