Update class name to get rid of reference to elasticsearch extensions

Open Beginner friendly
#199 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
70/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Quiet
Tech stack
typescript, wordpress
Domain
frontend, web-dev

Research direction

Start with plugin-templates/blocks/theme-faceted-search-facets/edit.tsx at the fieldset referenced in the issue. Review its class name and update it so the generated WordPress project has no Elasticsearch-related reference and follows the intended BEM structure; verify the resulting template markup.

Written by the indexing model from the issue text.

Description

Description

I would like to remove (or modify) the class name from the fieldset in the faceted search facets form to avoid improper BEM structure and unexpected references.

https://github.com/alleyinteractive/create-wordpress-project/blob/production/plugin-templates/blocks/theme-faceted-search-facets/edit.tsx#L19

Use Case

When a developer is creating a new WP project they shouldn't have to clean up code.

Dominant language
PHP
Stars
9
Forks
3
Avg merge
2d 20h
Merged PRs (30d)
4

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from alleyinteractive/create-wordpress-project

All issues in alleyinteractive/create-wordpress-project

Similar issues

More PHP issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.