Skip to content

Rename FB Product Sets to Facebook Product Sets and change location#2336

Merged
ibndawood merged 4 commits intofeature/hosted-woo-updatesfrom
add/2328-rename-fb-product-sets
Oct 21, 2022
Merged

Rename FB Product Sets to Facebook Product Sets and change location#2336
ibndawood merged 4 commits intofeature/hosted-woo-updatesfrom
add/2328-rename-fb-product-sets

Conversation

@ibndawood
Copy link
Copy Markdown
Contributor

@ibndawood ibndawood commented Oct 21, 2022

Changes proposed in this Pull Request:

Closes #2328

This PR:

  • Renames FB Product Sets to Facebook Product Sets.
  • Adds show_in_menu => false to register_taxonomy arguments. This would remove the Facebook Product Sets taxonomy from being displayed in Products > Facebook Product Sets.
  • Removes all references to FB Product Sets inside code comments.
  • Adds a private method to display the Product Sets menu item under Marketing. The method verifies if a connection has been established before adding the menu item under Marketing.
  • Adds a method that would filter the parent_file to correctly display the parent menu and submenu page when Facebook Product Sets is selected
  • Do the changed files pass phpcs checks? Please remove phpcs:ignore comments in changed files and fix any issues, or delete if not practical.

Screenshots:

The Facebook Product Set under Marketing Menu

image

The Facebook Product Set taxonomy page

image

Detailed test instructions:

  1. Verify Facebook for WooCommerce plugin is installed and activated.
  2. Verify that the store is not connected to Facebook yet.
  3. Verify there is no FB Product Sets menu item in the Products menu of WP admin.
  4. Verify there is no Facebook Product Sets menu item under the Marketing menu.
  5. Connect the store to Facebook.
  6. Verify the menu item Facebook Product Sets is present under the Marketing menu.
  7. Verify there is no Facebook Product Sets menu item under the Products menu.
  8. Click on the Facebook Product Sets menu and verify the taxonomy is referred to as Facebook Product Set in singular and plural wherever applicable.
  9. Verify the correct parent menu and submenu is selected when viewing Facebook Product Set page.

Changelog entry

Update - FB Product Set name changed to Facebook Product Set
Add - Facebook Product Set under the Marketing menu

@github-actions github-actions bot added changelog: add A new feature, function, or functionality was added. type: enhancement The issue is a request for an enhancement. labels Oct 21, 2022
@ibndawood ibndawood marked this pull request as ready for review October 21, 2022 12:55
@ibndawood ibndawood self-assigned this Oct 21, 2022
@ibndawood ibndawood requested a review from a team October 21, 2022 13:01
Copy link
Copy Markdown
Contributor

@rawdreeg rawdreeg left a comment

Choose a reason for hiding this comment

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

@ibndawood
Copy link
Copy Markdown
Contributor Author

ibndawood commented Oct 21, 2022

Thank you @rawdreeg for the review. I can see I missed the lower cases. I'll fix this and shall merge. Thank you for catching this.

@ibndawood ibndawood merged commit acc0011 into feature/hosted-woo-updates Oct 21, 2022
@ibndawood ibndawood deleted the add/2328-rename-fb-product-sets branch October 21, 2022 16:16
@ibndawood ibndawood mentioned this pull request Oct 27, 2022
@ibndawood ibndawood mentioned this pull request Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: add A new feature, function, or functionality was added. type: enhancement The issue is a request for an enhancement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants