Skip to content

Best Practices Applied #17

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 12 commits into from
Nov 4, 2021
Merged

Best Practices Applied #17

merged 12 commits into from
Nov 4, 2021

Conversation

realstoman
Copy link
Owner

  • Components best practices
  • Folder structure of components changed (single folder now)
  • Hire me modal components
  • Contact form component added
  • Added App prefix to shared components
  • Projects contents are dynamic
  • Pass in data through props

@netlify
Copy link

netlify bot commented Nov 4, 2021

✔️ Deploy Preview for vuejs-tailwindcss-portfolio ready!

🔨 Explore the source changes: 509a9f6

🔍 Inspect the deploy log: https://app.netlify.com/sites/vuejs-tailwindcss-portfolio/deploys/6184531feb7e4a00077a8198

😎 Browse the preview: https://deploy-preview-17--vuejs-tailwindcss-portfolio.netlify.app/

@realstoman realstoman merged commit 01731cc into main Nov 4, 2021
@realstoman realstoman deleted the components-bps branch October 2, 2022 07:39
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