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.

Sign up for GitHub

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

Jump to bottom

Open search user groups #3494

Merged
merged 4 commits into from
Dec 10, 2024
Merged

Open search user groups #3494

merged 4 commits into from
Dec 10, 2024

Conversation

kaimmej
Copy link
Contributor

@kaimmej kaimmej commented Dec 10, 2024

Description

Usergroups added

  • Added a usergroups page.
  • Decision - whether or not to add a usergroups collection. Instead of a collection I created a datafile that holds all of the page content and page images, as well as an array of usergroup groups.
  • Datafile created for usergroups, as well as all styling and content on the page.
  • Usergroup images and links extracted from Meetup and used on our site.
  • Navbar - usergroups link in the community dropdown menu changed to the “/usergroups/’ instead of to the meetup.

image

Issues Resolved

#3187

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.

@kaimmej
Added usergroups page and usergroups data
d30cf96 
Datafile created
Page and index created
Styling created
initial layout.

Signed-off-by: kaimmej <[email protected]>
@kaimmej
card image adjusted
7dcd7bb 
Signed-off-by: kaimmej <[email protected]>
@kaimmej
Usergroups button and new UG images
ad60ed0 
Used condensed images because page load was slow.
Added user group purple button.

Signed-off-by: kaimmej <[email protected]>
@kaimmej
uncomment site maps
c869245 
Signed-off-by: kaimmej <[email protected]>
@krisfreedain
Copy link
Member

Excellent. Looking at this now

krisfreedain
krisfreedain approved these changes Dec 10, 2024
View reviewed changes
Copy link
Member

@krisfreedain krisfreedain left a comment

Choose a reason for hiding this comment

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

This looks fantastic! Thank you @kaimmej

@krisfreedain krisfreedain merged commit 8620c42 into opensearch-project:main Dec 10, 2024
5 checks passed
@krisfreedain krisfreedain mentioned this pull request Dec 10, 2024
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krisfreedain krisfreedain krisfreedain approved these changes

@elfisher elfisher Awaiting requested review from elfisher elfisher is a code owner

@AMoo-Miki AMoo-Miki Awaiting requested review from AMoo-Miki AMoo-Miki is a code owner

@nknize nknize Awaiting requested review from nknize nknize is a code owner

@peterzhuamazon peterzhuamazon Awaiting requested review from peterzhuamazon peterzhuamazon is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE is a code owner

@dtaivpp dtaivpp Awaiting requested review from dtaivpp dtaivpp is a code owner

@kolchfa-aws kolchfa-aws Awaiting requested review from kolchfa-aws kolchfa-aws is a code owner

@nateynateynate nateynateynate Awaiting requested review from nateynateynate nateynateynate is a code owner

@natebower natebower Awaiting requested review from natebower natebower is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kaimmej @krisfreedain