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

Development #30

Merged
merged 36 commits into from
Oct 13, 2023
Merged

Development #30

merged 36 commits into from
Oct 13, 2023

Conversation

timDeHof
Copy link
Contributor

merge latest version of development branch to main

joekotvas and others added 30 commits October 8, 2023 22:16
feat(eslintrc.cjs): add support for YAML files
chore(commitlint.config.js): add commitlint configuration
chore(package.json): add commit script
chore(App.jsx): refactor App component to use Nav component
chore(posts.js): add getPost function to fetch individual post details
chore(main.jsx): remove unused imports
feat(Home.jsx): create Home component
feat(PostDetail.jsx): create PostDetail component
feat(Posts.jsx): create Posts component
feat(index.jsx): create Nav component to handle routing

The eslint configuration has been updated to include support for the latest ECMAScript version and module syntax. The ignore patterns have been modified to exclude additional files and directories. The rules for YAML files have been added to the eslint configuration.

A commitlint configuration file has been added to enforce conventional commit messages.

A commit script has been added to the package.json file to
…nt-config

Put quotes around linebreak-style value
…mits to be more informative

This will improve the teams' commits to the repo
… ones

added changelog using standard-version package
Add mirageJS integration and mock placeholder data
refactor(posts): update import paths for get-posts and use-posts
refactor(posts/components/post-card): update import path for react-router-dom
refactor(posts/components/post-list): update import path for post-card component
refactor(recipes/components/recipe-card): remove unused import
refactor(recipes/components/recipe-list): remove unused import
refactor(ui/Header): update import path for navLinks constant
refactor(pages/About): remove unused import
refactor(pages/PostDetail): update import path for GetPostById
refactor(pages/Posts): update import path for PostList
feat(pages/Recipes): add new RecipesPage component
refactor(pages/index): update import path for Recipes
refactor(routes/index): update import paths for pages and features/ui
refactor(vite.config): add aliases for src directory and its subdirectories

The changes were made to update import paths and add
…include a commit method

Also include is a method to bump up version releases
…ak-style--rule

Update linebreak-style to default to unix
timDeHof and others added 6 commits October 12, 2023 10:25
getting a failure to load config from `/opt/build/repo/vite.config.js` and saying 'build.command'
failed

BREAKING CHANGE: the netlify git hooks fail
Feat/impromate browser router
got netlify git hook to pass by deleting node_modules and package-lock.json. then reinstalling with `npm install`
@netlify
Copy link

netlify bot commented Oct 12, 2023

Deploy Preview for yumyumyes ready!

Name Link
🔨 Latest commit 52d97da
🔍 Latest deploy log https://app.netlify.com/sites/yumyumyes/deploys/65280fec509e840008ef889b
😎 Deploy Preview https://deploy-preview-30--yumyumyes.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@netlify
Copy link

netlify bot commented Oct 12, 2023

Deploy Preview for yumyumyesdev ready!

Name Link
🔨 Latest commit 52d97da
🔍 Latest deploy log https://app.netlify.com/sites/yumyumyesdev/deploys/65280fec8c143800076d3ae7
😎 Deploy Preview https://deploy-preview-30--yumyumyesdev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@github-actions
Copy link

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 0 0
Ignored 0 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

Copy link
Contributor

@DarrickFauvel DarrickFauvel left a comment

Choose a reason for hiding this comment

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

This runs fine for me. No eslint warnings (red squiggles) when I open a file on a Windows dev machine.

However, there is a very minor error in the browser console stating that the vite.svg is not found because it has moved to the src/assets folder.

@timDeHof timDeHof merged commit 99db787 into main Oct 13, 2023
11 checks 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.

3 participants