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

Allow to search by "Name" on the product attribute page (admin area) #7518

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

chrisjainsley
Copy link

@chrisjainsley chrisjainsley commented Jan 25, 2025

  • Updated IProductAttributeService to include an optional name parameter in GetAllProductAttributesAsync.
  • Modified ProductAttributeService to filter product attributes by name using the new parameter.
  • Added a new locale resource for product attribute name search in the XML resource file.
  • Enhanced ProductAttributeSearchModel with a SearchProductAttributeName property.
  • Updated List.cshtml to include a search block for filtering product attributes by name.

#7515

- Updated `IProductAttributeService` to include an optional `name` parameter in `GetAllProductAttributesAsync`.
- Modified `ProductAttributeService` to filter product attributes by name using the new parameter.
- Added a new locale resource for product attribute name search in the XML resource file.
- Enhanced `ProductAttributeSearchModel` with a `SearchProductAttributeName` property.
- Updated `List.cshtml` to include a search block for filtering product attributes by name.
- Made minor formatting adjustments for improved code readability.
Updated LINQ query formatting in the ProductAttributeService class to enhance code readability. Adjusted indentation for `orderby`, `where`, and `select` clauses. Also, modified a comment in the `GetProductAttributeByIdsAsync` method for consistent formatting.
@chrisjainsley chrisjainsley changed the title Add product attribute name filtering functionality Allow to search by "Name" on the product attribute page (admin area) Jan 25, 2025
@chrisjainsley chrisjainsley marked this pull request as ready for review January 25, 2025 01:01
@AndreiMaz AndreiMaz requested a review from exileDev January 25, 2025 04:48
@AndreiMaz AndreiMaz added this to the Version 4.90 milestone Jan 25, 2025
Added a hint for "SearchProductAttributeName" in
`LocalizationMigration.cs` and created a new locale
resource entry in `defaultResources.nopres.xml`.
Updated the HTML structure in `List.cshtml` for better
layout and user interaction, including a restructured
search block and improved button placements.
JavaScript functionality for deleting selected attributes
remains intact and has been reformatted for clarity.
Copy link

@qasas145 qasas145 left a comment

Choose a reason for hiding this comment

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

excellent

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.

4 participants