Skip to content

Added SOP for Display Non-Existent Pages in the Hamburger Menu #1221

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

Open
wants to merge 8 commits into
base: everything-bagel-pub
Choose a base branch
from

Conversation

swati1717
Copy link
Contributor

  • Added SOP for Display Non-Existent Pages in the Hamburger Menu
  • Added link to the summary.

Added SOP to show non-existent pages in the hamburger menu.
Added link for Display Non-Existent Pages in the Hamburger Menu
@swati1717
Copy link
Contributor Author

Updated changes as commented in the PR.

- [`ShopifyData.xml`](https://git.hotwax.co/commerce/oms/-/blob/develop/applications/shopify-connector/data/ShopifyData.xml)
- [`HamburgerMenusData.xml`](https://git.hotwax.co/commerce/oms/-/blob/develop/applications/hwmapps/data/commerce/HamburgerMenusData.xml)

Ensure that the required object info exists in these files.
Copy link
Contributor

Choose a reason for hiding this comment

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

what is the required object info?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added relevant information to the document.

```

#### Create Content and Link to Menu
Add the content details and associate the page with the required menu:
Copy link
Contributor

Choose a reason for hiding this comment

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

do we have more than one menu? How many menu options are there? where do I get valid menu ids?


### Step 3: Define Permissions
#### Create a Security Permission
Define the necessary permissions for page access:
Copy link
Contributor

Choose a reason for hiding this comment

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

why do I need to add security permissions?

Copy link
Contributor

Choose a reason for hiding this comment

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

what happens if I don't?

```

#### Assign Permission to Security Group
Grant access to the appropriate user group:
Copy link
Contributor

Choose a reason for hiding this comment

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

are there any suggested groups that I should add the permission to?

#### Create a Security Permission
Define the necessary permissions for page access:
```xml
<SecurityPermission description="View Shopify Solr Search Page"
Copy link
Contributor

Choose a reason for hiding this comment

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

how do I make sure that my permission gets categorized correctly in the users app?

@dt2patel dt2patel self-requested a review February 26, 2025 10:45
Copy link
Contributor

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

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

see comments

@swati1717
Copy link
Contributor Author

Updated changes as commented

@swati1717 swati1717 requested a review from dt2patel April 9, 2025 06:35
Copy link
Contributor

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

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

This document needs more content around what the main entities are for this operation, what is primary content id, how can user acquire them etc. Really think about this like you don't know ofbiz cms model and then try to use this doc for making a page other than the SOLR page.

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

Successfully merging this pull request may close these issues.

2 participants