Skip to content

Complete Challenge 1 and Challenge 2 - #40

Open
Daniel-pacifica wants to merge 11 commits into
masterfrom
feature/daniel-pulgarin
Open

Complete Challenge 1 and Challenge 2#40
Daniel-pacifica wants to merge 11 commits into
masterfrom
feature/daniel-pulgarin

Conversation

@Daniel-pacifica

Copy link
Copy Markdown

Summary

Completed Challenge 1 and Challenge 2.

Challenge 1

  • Built a custom AirPnP-style front page based on the provided above-the-fold reference.
  • Added a dedicated front-page template with a minimal custom header/footer.
  • Used Tailwind utility classes for layout, spacing, typography, and responsive behavior.
  • Included a MeanPug pug asset as part of the hero/avatar treatment.
  • Added defensive checks around optional ACF dependencies in the base theme.

Challenge 2

  • Created a must-use core plugin for the law firm content model.
  • Registered custom post types for Attorneys, Practice Areas, Case Results, Testimonials, Locations, and FAQs.
  • Registered law firm taxonomies for Practice Area Types, Attorney Roles, Areas Served, and Case Result Types.
  • Added native WordPress post meta definitions with sanitization, auth callbacks, and REST support.
  • Added optional ACF local field groups for editorial UI when ACF is available.
  • Added helper functions for common relationships and structured data needs.

Notes

The base theme referenced ACF option fields in a few places, so I added defensive checks to avoid fatal errors in a clean Docker/WordPress setup without requiring additional plugin installation.

I also added a local Webpack config referenced by the existing Docker static service so the watcher can run correctly during local development.

@Daniel-pacifica

Copy link
Copy Markdown
Author

Hi! thanks again for the opportunity. I completed Challenge 1 and Challenge 2 and opened this PR for review.

Regards!

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