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

Merge develop into release #447

Merged
merged 4 commits into from
Aug 16, 2021
Merged

Merge develop into release #447

merged 4 commits into from
Aug 16, 2021

Conversation

StanislavKucher
Copy link
Contributor

No description provided.

KuzmaJava and others added 4 commits August 16, 2021 14:17
* Edited findAllByAuthorIdAndStatus method to findAllByAuthorIdAndStatusOrderByPublishedAtDesc in the PostRepository to sort by descending.

* The tests were edited
…stServiceImpl class (#443)

Adjusted SQL queries per each section by adding to ORDER BY clause ordering by post ID DESC
* add application-prod.properties

* add V13.1__add_demo_data.sql

* add posts to the V13.1__add_demo_data.sql

* add has_posts all true to the V13.1__add_demo_data.sql

* rename V13.1__add_demo_data.sql to 14.1
@StanislavKucher StanislavKucher added bug Something isn't working backend Tasks for Java team SQL SQL issues labels Aug 16, 2021
@StanislavKucher StanislavKucher added this to the Sprint 19 milestone Aug 16, 2021
@StanislavKucher StanislavKucher requested a review from Izarx August 16, 2021 18:20
@StanislavKucher StanislavKucher self-assigned this Aug 16, 2021
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@StanislavKucher StanislavKucher merged commit dc55782 into release Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Tasks for Java team bug Something isn't working SQL SQL issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants