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

Update admin.phtml - Added Nonce to inline javascript #535

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timpea-splat
Copy link
Contributor

No description provided.

@mfilip
Copy link
Member

mfilip commented Jun 18, 2024

We will be adding that PR #536 instead

@timpea-splat
Copy link
Contributor Author

Could you add the following to the admin.phtml?

autocompleteOverride: autocompleteOverride,

I have the following override to remove the organisation from the 1st line of the address

{ "removeOrganisation": true }

Or could you add an Admin setting for removeOrganisation? So it is easy to set.

Without it, the organisation is always repeated in the 1st line of the address.

@mfilip
Copy link
Member

mfilip commented Jun 19, 2024

@timpea-splat think we can add it in different way because override can cause problems for other users. Think best way will be adding that option as checkbox to select and set after.

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

Successfully merging this pull request may close these issues.

2 participants