Skip to content

AYS | Project Package/Layer/Infrastructure Renew #323

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 52 commits into from
Jun 24, 2024
Merged

AYS | Project Package/Layer/Infrastructure Renew #323

merged 52 commits into from
Jun 24, 2024

Conversation

agitrubard
Copy link
Collaborator

@agitrubard agitrubard commented Jun 23, 2024 •
edited by yumerchalashkan
Loading

Checklist

Before submitting your pull request, ensure the following:

  • Title and Branch Naming Conventions:

  • Local Testing:

    • I have tested my changes locally on Postman, and they are working as expected.

  • Code Quality:

    • The code is formatted according to the project's coding guidelines and style.
    • The code has been reviewed to ensure its quality.
    • The code does not contain any issues flagged by SonarLint.

  • Documentation:

    • Necessary documentation has been added or existing documentation has been updated, specifically detailing changes made in Postman.

  • Testing:

    • Relevant unit tests have been written and included.
    • Relevant integration tests have been written and included.

  • Reviewers and Assignees:

    • Default reviewers have been assigned to this pull request.
    • Assignees have been added if necessary.

  • Labels and Associations:

    • No specific actions are required in the Labels and Associations section for this pull request.

agitrubard added 30 commits June 12, 2024 17:13
@agitrubard
AYS-275 | auth.model.dto.request Package Has Been Moved to `auth.mo…
04eb5e3 
…del` Package
@agitrubard
AYS-275 | auth.model.dto.response Package Has Been Moved to `auth.m…
226ed53 
…odel` Package
@agitrubard
AYS-275 | common.model.dto.request Package Has Been Moved to `commo…
0116a63 
…n.model` Package
@agitrubard
AYS-275 | common.model.dto.response Package Has Been Moved to `comm…
295bd62 
…on.model` Package
@agitrubard
AYS-275 | AysErrorResponse Class Has Been Moved to `common.model.re…
46a03b3 
…sponse` Package
@agitrubard
Merge remote-tracking branch 'origin/main' into refactor/AYS-275_package
4fe273a
@agitrubard
Merge remote-tracking branch 'origin/main' into refactor/AYS-275_package
7e19e3f 
# Conflicts:
#	src/test/java/org/ays/emergency_application/controller/EmergencyEvacuationApplicationControllerTest.java
#	src/test/java/org/ays/emergency_application/controller/EmergencyEvacuationApplicationSystemTest.java
@agitrubard
AYS-275 | Descriptions of Lombok Annotation Have Been Removed from Ja…
f253733 
…vadoc in `AysTokenResponse` Class
@agitrubard
AYS-275 | institution.model.dto.response Package Has Been Moved to …
ce080e8 
…`institution.model` Package
@agitrubard
AYS-275 | user.model.dto.request Package Has Been Moved to `user.mo…
cc9a2d8 
…del` Package
@agitrubard
AYS-275 | user.model.dto.response Package Has Been Moved to `user.m…
7e3b691 
…odel` Package
@agitrubard
AYS-275 | user Package Have Been Merged with auth Package
83aa814
@agitrubard
AYS-275 | Imports Have Been Fixed in Test Classes
a4c8b5a
@agitrubard
AYS-275 | Port/Adapter Layer Has Been Created and Integrated to Par…
3ec8dcb 
…ameter Service Implementation
@agitrubard
AYS-275 | Parameter Adapter Layer Has Been Covered with Unit Tests
720d338
@agitrubard
AYS-275 | AbstractRestControllerTest.initializeAuth() Method Has Be…
14cb4e5 
…en Refactored
@agitrubard
AYS-275 | Parameter Service Layer Has Been Covered with Unit Tests
ca2291a
@agitrubard
AYS-275 | AysParameterService.getParameters() Method Has Been Renam…
09abb12 
…ed as `findAll()`
@agitrubard
AYS-275 | AysParameterService.getParameters() Method Has Been Renam…
144e318 
…ed as `findByName()`
@agitrubard
AYS-275 | Port/Adapter Layer Has Been Created and Integrated to Ins…
5fc32a6 
…titution Service Implementation
@agitrubard
AYS-275 | Institution Service Layer Has Been Covered with Unit Tests
f95bbc7
@agitrubard
AYS-275 | Institution Adapter Layer Has Been Covered with Unit Tests
e59cd08
@agitrubard
AYS-275 | Javadoc Has Been Updated in InstitutionReadPort Class
402e866
@agitrubard
AYS-275 | existsActiveById Method Has Been Removed from `Institutio…
66e02fa 
…nRepository` and Method Has Been Added to `InstitutionAdapter`
@agitrubard
AYS-275 | emergency_application.model.dto.request Package Has Been …
c8a1f67 
…Moved to `emergency_application.model` Package
@agitrubard
AYS-275 | emergency_application.model.dto.response Package Has Been…
823fda4 
… Moved to `emergency_application.model` Package
@agitrubard
AYS-275 | Auth v1 Has Been Removed
6ae2ab5
@agitrubard
AYS-275 | Documentation of Package and Layer Infrastructure Has Been …
7dca6a8 
…Updated
@agitrubard
AYS-275 | Postman URLs Have Been Updated
481771b
@agitrubard
AYS-275 | Infrastructure of Project, Class Names, Package Names Have …
8f4fa05 
…Been Updated, Unnecessary Codes Have Been Removed, Port/Adapter Layer Has Been Integrated and All Tests Have Been Refactored
@agitrubard agitrubard requested a review from skayikci as a code owner June 23, 2024 14:05
@agitrubard agitrubard changed the title Refactor/ays 275 package AYS | Project Package/Layer/Infrastructure Renew Jun 23, 2024
agitrubard added 20 commits June 23, 2024 22:57
@agitrubard
AYS-275 | CONTRIBUTING.md File Has Been Fixed
9dc6b9c
@agitrubard
AYS-275 | EmergencyEvacuationApplicationRepositoryTest Class Has Be…
81d42af 
…en Removed
@agitrubard
AYS-275 | Package Infrastructure Example Image Has Been Fixed
3f7cc4e
@agitrubard
AYS-275 | .github/ISSUE_TEMPLATE Package Has Been Removed Because I…
93200b1 
…t's Unused
@agitrubard
AYS-275 | Common Maven Errors Header and Content Has Been Updated
8720e02
@agitrubard
AYS-275 | Unused Parameter Has Been Removed from 2-ays-dml.xml File
5fdb205
@agitrubard
AYS-275 | Nested Classes Have Been Renamed in AysValidTestData Class
1476352
@agitrubard
AYS-275 | UserV2Builder Has Been Renamed as AysUserBuilder
5780c09
@agitrubard
AYS-275 | AysLoginRequestV2Builder Has Been Renamed as `AysLoginReq…
71a8945 
…uestBuilder`
@agitrubard
AYS-275 | UserEntityV2Builder Has Been Renamed as `AysUserEntityBui…
b3d7b4c 
…lder`
@agitrubard
AYS-275 | AysAuthControllerV2Test Has Been Renamed as `AysAuthContr…
45ad86b 
…ollerTest`
@agitrubard
AYS-275 | PermissionBuilder Has Been Renamed as AysPermissionBuilder
84c7b54
@agitrubard
AYS-275 | test/emergency_application/model/dto/request Package Has …
1bfde03 
…Been Moved to `test/emergency_application/model` Package
@agitrubard
AYS-275 | test/emergency_application/model/dto/response Package Has…
dd690ed 
… Been Moved to `test/emergency_application/model` Package
@agitrubard
AYS-275 | EmergencyEvacuationApplicationListRequestBuilder Class Ha…
35700c2 
…s Been Moved to `test/emergency_application/model/request` Package
@agitrubard
AYS-275 | RoleCreateRequestBuilder Has Been Renamed as `AysRoleCrea…
c6df398 
…teRequestBuilder`
@agitrubard
AYS-275 | PermissionEntityBuilder Has Been Renamed as `AysPermissio…
4cd35c9 
…nEntityBuilder`
@agitrubard
AYS-275 | RoleEntityBuilder Has Been Renamed as AysRoleEntityBuilder
54b60ec
@agitrubard
AYS-275 | test/common/model/dto/request Package Has Been Moved to `…
d602033 
…test/common/model` Package
@agitrubard
AYS-275 | Javadoc of BaseMapper Class Has Been Fixed
290d63e
@agitrubard agitrubard merged commit 344f11d into main Jun 24, 2024
@agitrubard agitrubard deleted the refactor/AYS-275_package branch June 24, 2024 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hsrtcskn hsrtcskn Awaiting requested review from hsrtcskn

@egehanasal egehanasal Awaiting requested review from egehanasal egehanasal is a code owner

@ibrahimdenizz ibrahimdenizz Awaiting requested review from ibrahimdenizz

@fatihdas fatihdas Awaiting requested review from fatihdas

@hmzshin hmzshin Awaiting requested review from hmzshin

@SuedaSen SuedaSen Awaiting requested review from SuedaSen

@MenekseYuncu MenekseYuncu Awaiting requested review from MenekseYuncu

@yumerchalashkan yumerchalashkan Awaiting requested review from yumerchalashkan

@Cengizhan1 Cengizhan1 Awaiting requested review from Cengizhan1

@AbdullahCelikcode AbdullahCelikcode Awaiting requested review from AbdullahCelikcode

@elifintizamoglu elifintizamoglu Awaiting requested review from elifintizamoglu

@melihcoskun melihcoskun Awaiting requested review from melihcoskun

@ademk7604 ademk7604 Awaiting requested review from ademk7604

@anilhazar anilhazar Awaiting requested review from anilhazar

@SaidMuratOzdemir SaidMuratOzdemir Awaiting requested review from SaidMuratOzdemir

@skayikci skayikci Awaiting requested review from skayikci skayikci is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@agitrubard