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

Fixes to pages #8

Merged
merged 3 commits into from
Dec 7, 2024
Merged

Fixes to pages #8

merged 3 commits into from
Dec 7, 2024

Conversation

dvidsilva
Copy link
Contributor

@dvidsilva dvidsilva commented Dec 7, 2024

This pull request includes several updates to the codebase, primarily focusing on adding a new Subscribe component, improving type definitions, and enhancing SEO capabilities.

New Component Addition:

  • src/components/Subscribe.tsx: Introduced a new Subscribe component that allows users to subscribe to a newsletter. It includes form handling and integration with the markketplace configuration for API calls.

Type Definitions and Usage:

Configuration Enhancements:

  • src/config.ts: Added a colors object to the markketplace configuration to store primary and accent colors.

SEO Improvements:

New Page Implementation:

  • src/pages/newsletter.astro: Created a new newsletter page that uses the Subscribe component and includes layout and SEO configurations.This pull request includes changes to the src directory to improve type safety and consistency across components and pages. The most important changes include the introduction of a new type alias for activeNav, updates to the Header component, and enhancements to the TagPosts layout.

Type safety and consistency improvements:

Enhancements to TagPosts layout:

  • src/layouts/TagPosts.astro: Added the SEO property to the data object being passed, ensuring that SEO metadata is included in the rendered output.

@dvidsilva dvidsilva merged commit ecde144 into main Dec 7, 2024
1 check 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.

1 participant