Skip to content

AI agents page #15097

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

Merged
merged 13 commits into from
Mar 14, 2025
Merged

AI agents page #15097

merged 13 commits into from
Mar 14, 2025

Conversation

corwintines
Copy link
Member

@corwintines corwintines commented Mar 13, 2025

- Introduced a new navigation item for AI agents in the useNav hook.
- Added corresponding translations for AI agents and their descriptions in common.json.
- Updated template-usecase.json to include AI agents in the dropdown options.
- Enhanced UseCases layout to link to the AI agents page with tracking events.
@github-actions github-actions bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels Mar 13, 2025
Copy link

netlify bot commented Mar 13, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 26174e3
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/67d4506881266a0008260e61
😎 Deploy Preview https://deploy-preview-15097--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 45 (🔴 down 12 from production)
Accessibility: 95 (🟢 up 1 from production)
Best Practices: 87 (🔴 down 8 from production)
SEO: 98 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

pettinarip
pettinarip previously approved these changes Mar 13, 2025
Copy link
Member

@pettinarip pettinarip left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Member

Choose a reason for hiding this comment

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

Dont have a better solution atm with the current setup but I don't like the direction of these very specific components that we can't reuse and that don't share/reuse other existing styles or components.

Was thinking that maybe we could fix this in the md files itself. Where you can handle translations and images. Something like

<Boxes>
  <Box>
    ![AI16Z](./ai16z.png)
    
    blabla
  </Box>
  <Box>
    ![GAME](./game.png)
    
    blabla
  </Box>
</Boxes>

Unfortunately, this is not supported in the current mdx version (mdx syntax inside jsx) but will be possible after we merge #14968 ! since we are also upgrading the mdx version.

<div>{title}</div>
<div className="mb-0 text-sm opacity-60">{description}</div>
<div className="mb-0 flex flex-col gap-2 text-sm opacity-60">
{contentItems}
Copy link
Member

Choose a reason for hiding this comment

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

Given that contentItems is optional, shouldn't we consider that case and avoid rendering the wrapping div?

@konopkja
Copy link
Contributor

Screenshot 2025-03-13 at 21 27 06

the text and button here for some reason looks disabled @corwintines

@corwintines corwintines merged commit 2acc7f9 into staging Mar 14, 2025
6 of 10 checks passed
@corwintines corwintines deleted the ai-page branch March 14, 2025 17:38
This was referenced Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants