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

Fix Posts #93

Merged
merged 3 commits into from
Jun 15, 2024
Merged

Fix Posts #93

merged 3 commits into from
Jun 15, 2024

Conversation

kimchanhyung98
Copy link
Owner

@kimchanhyung98 kimchanhyung98 commented Jun 15, 2024

작업 내용

  • 게시판 코드 개선
  • Invoke Controller 분리
  • transaction 제거

@kimchanhyung98 kimchanhyung98 changed the base branch from develop to renewal June 15, 2024 10:28
@kimchanhyung98 kimchanhyung98 merged commit 2e86fce into renewal Jun 15, 2024
2 of 7 checks passed
@kimchanhyung98 kimchanhyung98 deleted the fix-posts branch June 15, 2024 10:28
Copy link

sonarcloud bot commented Jun 15, 2024

Quality Gate Passed Quality Gate passed

Issues
19 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

kimchanhyung98 added a commit that referenced this pull request Jun 19, 2024
* Fix Users (#92)

* Add laravel-shift/blueprint

* Create blueprint(draft)

* Remove Account dir (update User)

* Update User controllers

* Fix styling

---------

Co-authored-by: kimchanhyung98 <kimchanhyung98@users.noreply.github.com>

* Fix Posts (#93)

* Update Invoke Controllers

* Update Requests,Resources

* Remove unused files

* Add Comments (#94)

* Update post show

* Create Comment

* Update routes api

* Add �Likes (#95)

* Create Like Controllers

* Create Like Request,Resource

* Update api.php

* Bump dedoc/scramble from 0.10.11 to 0.10.12 (#90)

Bumps [dedoc/scramble](https://github.com/dedoc/scramble) from 0.10.11 to 0.10.12.
- [Release notes](https://github.com/dedoc/scramble/releases)
- [Commits](dedoc/scramble@v0.10.11...v0.10.12)

---
updated-dependencies:
- dependency-name: dedoc/scramble
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: kimchanhyung98 <kimchanhyung98@gmail.com>

* Update composer.lock

* Fix larastan errors

* Fix testing (#96)

* Add Tests

* Remove unused files

* Update User controllers

* Update User Tests

* Create factroies

* Create seeders

* Fix typo

* Update User tests

* Fix typo

* Fix typo

* Create Post Tests

* Update http methods

* Update PostFactory.php

* Update Like tests

* Update PostShowTest.php

* Fix pint

* Update pull_request_template.md

* Update composer.json

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: kimchanhyung98 <kimchanhyung98@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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