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

[FEATURE] Partner filtering. Collection renamed to SolutionsProviders #3473

Merged
merged 42 commits into from
Dec 4, 2024

Conversation

kaimmej
Copy link
Contributor

@kaimmej kaimmej commented Nov 27, 2024

Description: Summary of changes

Solutions providers update.

  • Added filtering
  • renamed _partners/ collection to _solutionsProvider
  • cleaned backend metadata for partners

Desktop view of solutions provider filtering
image

Mobile view of solutions provider filtering
image

Individual solutions providers page has been revamped
image

Description: Detailed description of changes

Renamed collection: now solutions providers.

  • Style sheet was renamed from _partners.scss to _solutionsProviders.scss. The assets/css/output.scss was changed to input the renamed style sheet.
  • The collection was renamed from _partners/ to _solutionsProviders/.
  • Content in the [sample.md](http://sample.md) files were changed to reflect this change.
  • Image pathing for partners stays the same! this was too big of a change. it would have meant updating the image pathing on all existing partners, and renaming the assets/img/partners folder. It remains the same.
  • in order to do this, the class styling needed to be changed.
  • Update to the pathing in footer and navbar - now links to the new solutionsProviders page instead of the non-existant partners page. This update took place in their respective datafiles.
  • Renamed the navbar ‘partners’ link in the community dropdown to ‘providers’

Collection - specific detail page updates:

  • changed the individual partner.html to solutionsProviders.html, and updated the config.yml to update the default solutionsProviders page to this layout.

Github Issue templates

  • renamed to solutions providers.

Metadata, update and clean the categories for SolutionsProviders

  • Updated the metadata within business_type, opensearch_tech and regions to the finalized list of categories.
  • Updated all solutions providers to this finalized list.
  • updated the [sample.md](http://sample.md) files to reflect this updated list.
  • updated the github issue templates to reflect this updated list.

Partner filtering:

  • created new filterbar (mobile and desktop) which is rendered from a datafile (page_solutionProviders_datafile.json.
  • on pageload, javascript pulls the entire collection of partners into local memory, and applies the attribute categories that we will filter on. OnClick methods are applied to the buttons, and checkboxes that allow us to update the category and filter count metrics in real time. Finally, when ‘apply filters’ is clicked, the partners are filtered through the use of this object array that we created on page load.

Partner individual page

  • rebuilt template into a new design.

Issues Resolved

#3430
#1641

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Font awesome has been updated since these were added. Having them in the assets/css folder is the correct placement.

Signed-off-by: kaimmej <[email protected]>
Formatted the frontmatter field for the opensearch tech into an array, and gave consistent tags across the board.
Standardized spelling for tags to have capital first letter.

Signed-off-by: kaimmej <[email protected]>
Removed the brackets from the opensearch tech. Grouped and organized the search filters and tags together.

Signed-off-by: kaimmej <[email protected]>
created the scroll to top component, the javascript and css files.

Signed-off-by: kaimmej <[email protected]>
javascript turns it visible when you start to scroll. Finicky, still needs some work to get it working correctly.

Signed-off-by: kaimmej <[email protected]>
Built custom scss components - both an accordion and button. These custom components were placed in a new scss folder labeled '/components/'

Also, datafile was changed to put the Solutions Providers filtering categories, in alphabetical order.

Signed-off-by: kaimmej <[email protected]>
- Style sheet was renamed from _partners.scss to _solutionsProviders.scss. The assets/css/output.scss was changed to input the renamed style sheet.
- The collection was renamed from _partners/ to _solutionsProviders/.
- Collection
    - Content in the [sample.md](http://sample.md) files were changed to reflect this change.
- Image pathing for partners stays the same! this was too big of a change. it would have meant updating the image pathing on all existing partners, and renaming the assets/img/partners folder. It remains the same.
-
- in order to do this, the class styling needed to be changed.
- Update to the pathing in footer and navbar - now links to the new solutionsProviders page instead of the non-existant partners page. This update took place in their respective datafiles.
- Renamed the navbar ‘partners’ link in the community dropdown to ‘providers’

Signed-off-by: kaimmej <[email protected]>
Signed-off-by: kaimmej <[email protected]>
This was a vulnerability for XXS attacks and it was not being used on the site.

Signed-off-by: kaimmej <[email protected]>
@nateynateynate
Copy link
Member

Dude! This looks awesome so far! I'll file a few feature requests for some slight changes but I don't see anything that would prevent us from pushing this live. Merging to staging.

Copy link
Member

@nateynateynate nateynateynate left a comment

Choose a reason for hiding this comment

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

Pushing to staging for a better look.

@nateynateynate nateynateynate merged commit 4c11f20 into opensearch-project:main Dec 4, 2024
5 checks passed
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