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

update: frontend boilerplate #9

Merged
merged 12 commits into from
Nov 15, 2023
Merged

update: frontend boilerplate #9

merged 12 commits into from
Nov 15, 2023

Conversation

JLin-99
Copy link

@JLin-99 JLin-99 commented Nov 13, 2023

⚡ Frontend Boilerplate - ###

💻 What type of change is this?

  • 💎 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Styling
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build / CI

⭐ Description

I've updated the resource folder.

📷 Screenshots

Screenshot 2023-11-13 at 09 40 32
Screenshot 2023-11-13 at 09 39 11
Screenshot 2023-11-13 at 09 39 15
Screenshot 2023-11-13 at 09 39 19

✅ Checklist

  • This PR can be merged (it is not a draft, work in progress, or blocked on another PR)
  • I have tested this change locally in multiple screen sizes.

package.json Outdated
@@ -3,17 +3,22 @@
"scripts": {
"dev": "vite",
"build": "vite build",
"format": "prettier --check \"./resources/js/**/*.{js,cjs,mjs,ts,tsx,md,json}\" --ignore-path ../../.gitignore",
"format": "prettier --write \"./resources/js/**/*.{js,cjs,mjs,ts,tsx,md,json}\" --ignore-path ../../.gitignore",

Choose a reason for hiding this comment

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

the normal format script should be just --check because it's being called in the github actions and we DON'T want top modify our code there, only locally. Please add a different script that does the --write just so we are sure we are calling it only ever by hand

Copy link
Author

@JLin-99 JLin-99 Nov 14, 2023

Choose a reason for hiding this comment

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

sorry I forgot to remove it I was trying the --write option because my local prettier went crazy haha. That being said, should I make another script that does the --write or just put it back as it was before?

Choose a reason for hiding this comment

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

let's make another script

Copy link
Author

Choose a reason for hiding this comment

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

done

@JLin-99 JLin-99 merged commit 3074450 into 10.x Nov 15, 2023
2 checks passed
@JLin-99 JLin-99 deleted the bugfix/frontend-boilerplate branch November 15, 2023 15:37
sojeda pushed a commit that referenced this pull request Nov 28, 2023
* chore: app file rename

* chore: app file rename

* feat: frontend environment variables

* feat: update env.example

* feat: axios instances

* feat: authentication

* feat: add examples

* feat: update PR template

* fix: prettier

* fix: overflow scroll

* fix: .github/PULL_REQUEST_TEMPLATE.md

* feat: add format options
sojeda pushed a commit that referenced this pull request Jan 2, 2024
* chore: app file rename

* chore: app file rename

* feat: frontend environment variables

* feat: update env.example

* feat: axios instances

* feat: authentication

* feat: add examples

* feat: update PR template

* fix: prettier

* fix: overflow scroll

* fix: .github/PULL_REQUEST_TEMPLATE.md

* feat: add format options
sojeda pushed a commit that referenced this pull request Jan 2, 2024
* chore: app file rename

* chore: app file rename

* feat: frontend environment variables

* feat: update env.example

* feat: axios instances

* feat: authentication

* feat: add examples

* feat: update PR template

* fix: prettier

* fix: overflow scroll

* fix: .github/PULL_REQUEST_TEMPLATE.md

* feat: add format options
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