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

Moved keywords in language file #16061

Merged
merged 2 commits into from
Jan 13, 2025
Merged

Conversation

snipe
Copy link
Owner

@snipe snipe commented Jan 13, 2025

This just breaks out the invisible keywords that informs the admin settings search box, and adds barcodes to the list so folks can more easily find the barcodes settings since we merged those into labels.

Copy link

what-the-diff bot commented Jan 13, 2025

PR Summary

  • Improvement in the Language File Organization
    The PR has made the language file cleaner by eliminating specific keywords related to various settings, like general settings, PHP overview and more. This removal helps in keeping the file concise and less cluttered.

  • Introduction of a New 'keywords' Structure
    A much more orderly method is adopted to manage keywords associated with settings. A new 'keywords' array has been created in the language file to facilitate improved organization.

  • Updates in Settings View
    The settings view has been updated to work with the newly introduced 'keywords' structure instead of using separate individual keyword strings, which have now been removed. This makes referencing easier and more efficient.

  • Modification in Help Text for Label Settings
    There has been a slight adjustment in the help text provided for the label settings option. It now specifies "Barcodes & label settings" instead of just "Label sizes & settings." This provides a more comprehensive, clearer understanding for users.

@snipe snipe merged commit e32ed03 into develop Jan 13, 2025
9 checks passed
@snipe snipe deleted the moved_keywords_on_admin_settings branch January 13, 2025 17:10
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.

1 participant