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/coveo implementation #308

Merged
merged 9 commits into from
May 13, 2024
Merged

Conversation

ankitguptaGspann
Copy link
Collaborator

@ankitguptaGspann ankitguptaGspann commented May 9, 2024

Copy link

aem-code-sync bot commented May 9, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link

aem-code-sync bot commented May 9, 2024

Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

ankitguptaGspann and others added 8 commits May 9, 2024 22:36
Created New Search Block
Added search CSS in search.CSSS
Update the Header.js to add the new search input in header
Removed the old header code
Removed the old search CSS
Update the template.css code for header changes
Update aem.js for debounce functionality on input search
Update the header CSS in gcse.css
@ankitguptaGspann ankitguptaGspann force-pushed the feature/coveo-implementation branch from 0ece19b to ee479da Compare May 9, 2024 17:28
@aem-code-sync aem-code-sync bot temporarily deployed to feature/coveo-implementation May 9, 2024 17:28 Inactive
Copy link
Collaborator

@davenichols-DHLS davenichols-DHLS left a comment

Choose a reason for hiding this comment

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

From a coding perspective, I have reviewed and happy to sign off. I have also checked the UX/UI and this appears in good shape.

@aem-code-sync aem-code-sync bot temporarily deployed to feature/coveo-implementation May 10, 2024 15:15 Inactive
@davenichols-DHLS davenichols-DHLS merged commit bef9b20 into main May 13, 2024
2 checks passed
@davenichols-DHLS davenichols-DHLS deleted the feature/coveo-implementation branch May 13, 2024 13:14
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.

3 participants