Skip to content

AV-68536: complete quickstart code for spring data #34

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 41 commits into from
Feb 26, 2024
Merged

AV-68536: complete quickstart code for spring data #34

merged 41 commits into from
Feb 26, 2024

Conversation

teetangh
Copy link
Contributor

No description provided.

@teetangh teetangh closed this Dec 12, 2023
@teetangh teetangh deleted the AV-68536-java-quickstart-using-spring-data branch December 12, 2023 00:39
@teetangh teetangh restored the AV-68536-java-quickstart-using-spring-data branch December 12, 2023 00:42
@teetangh teetangh changed the title AV-68536: complete spring data couchbase quickstart AV-68536: legacy Dec 12, 2023
@teetangh teetangh reopened this Dec 12, 2023
@teetangh teetangh changed the title AV-68536: legacy AV-68536: complete quickstart for spring data Dec 12, 2023
@teetangh teetangh marked this pull request as draft December 12, 2023 00:44
@teetangh teetangh self-assigned this Dec 12, 2023
@teetangh teetangh changed the title AV-68536: complete quickstart for spring data AV-68536: complete quickstart code for spring data Dec 12, 2023
@teetangh
Update project settings and controller methods
d29e544 
Update AirlineRepository and AirlineService
@teetangh
Add direct connections query to AirportRepository and AirportService
407214c
@teetangh
Update .gitignore and remove unused settings.json file. Refactor repo…
96440d2 
…sitory annotations.
@teetangh
Added devtools,integration tests,model validation,
a460ae4
@teetangh
Refactor integration tests to use RestResponsePage
b557d2c
@teetangh
updated controllers with error handling + used lombok data and builde…
63bd9fa 
…r + refactored integration tests
@teetangh
Add Dockerfile and make repository classes final
496204d
@teetangh
Add direct connections API endpoint and update repository
722adb2
@teetangh
Remove unused variable in CouchbaseConfiguration.java
91a64f6
@teetangh
Refactor Application class with Spring Data and Swagger documentation
108a39d
@teetangh
Delete unused classes and update application.properties
557638a
@teetangh
Overridden Cluster connect method
d203c1b
@teetangh
Add .iml and target/ to .gitignore, handle exceptions in test setup a…
b9a4e54 
…nd teardown
@teetangh
Add summary annotations to list routes and list airports APIs
a4f9116
@teetangh
Remove unused healthcheck,index and profile
8df40d0
@teetangh
Docker finally working
ec1c651
@teetangh
Update GitHub Actions workflow and Couchbase configuration
bb27e97
@teetangh
Update GitHub Actions workflow to include JDK setup and cache configu…
ea4a31e 
…ration
teetangh added 13 commits January 10, 2024 16:12
@teetangh
Refactor integration tests to use dynamic base URI
1a3ceaa 
This commit refactors the integration tests in the `RouteIntegrationTest` and `AirlineIntegrationTest` classes to use a dynamic base URI based on the value of the `spring.couchbase.bootstrap-hosts` property. This allows the tests to run against different environments without modifying the code.
@teetangh
Update Gradle source and target compatibility to 1.8
30a7790
@teetangh
updated gradle commands
2456822
@teetangh
Update secrets in application.properties
ce972bf
@teetangh
Update secrets in application.properties
7bf04d4
@teetangh
Update secrets in application.properties file
9deb19f
@teetangh
update gradle secrets
2753a13
@teetangh
Update secrets in application.properties file
f8ec5a6
@teetangh
Refactor test setup and teardown methods
8e90d87
@teetangh
update github workflow yaml configuration and secrets
ffc20fa
@teetangh
Update secrets in application.properties
7a5b4a6
@teetangh
Refactor test classes and add logging statements
40a1999
@teetangh
Refactor model classes and update route integration test
f6cff34
@teetangh teetangh marked this pull request as ready for review January 10, 2024 18:53
@teetangh teetangh added the enhancement New feature or request label Jan 10, 2024
nithishr
nithishr requested changes Jan 17, 2024
View reviewed changes
Copy link

@nithishr nithishr left a comment

Choose a reason for hiding this comment

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

Only the README has been reviewed.
Recommendations:

  • Move the code explanations into the tutorial on Developer Portal
  • Adapt the text to be Capella first.
  • Check the recommendations from spring boot as well.

deniswsrosa
deniswsrosa requested changes Jan 31, 2024
View reviewed changes
Copy link
Contributor

@deniswsrosa deniswsrosa left a comment

Choose a reason for hiding this comment

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

couple of things needs to change

deniswsrosa
deniswsrosa requested changes Feb 21, 2024
View reviewed changes
Copy link
Contributor

@deniswsrosa deniswsrosa left a comment

Choose a reason for hiding this comment

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

There are a few adjusments left

@teetangh teetangh requested a review from deniswsrosa February 26, 2024 07:50
@deniswsrosa deniswsrosa merged commit 93917a2 into main Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deniswsrosa deniswsrosa deniswsrosa approved these changes

@nithishr nithishr Awaiting requested review from nithishr

Assignees

@teetangh teetangh

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@teetangh @deniswsrosa @nithishr