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

If anyone wants some production level code #35

Open
wants to merge 9 commits into
base: master2
Choose a base branch
from

Conversation

nikhilniky
Copy link

My changes /contributions

developed new logging
added PostgreSQL
fixed sonar issues
removed vulnerabilities
converted script for Postgresql

nikhilniky and others added 5 commits October 30, 2023 21:17
added logging

fixed sonar issues

refactoredcode and removed unwated code

updated SQL file for this
@jaygajera17
Copy link
Owner

Great ! , currently no maintainers are active so after reviewing your code i will merge in a week .

added logging

fixed sonar issues

refactoredcode and removed unwated code

updated SQL file for this
added logging

fixed sonar issues

refactoredcode and removed unwated code

updated SQL file for this
@nikhilniky
Copy link
Author

Thanks for quick response

@jaygajera17
Copy link
Owner

it is perfectly running ? if yes then i create another branch for postgresql because it is hard to check all your lots of changes.

@nikhilniky
Copy link
Author

nikhilniky commented Nov 18, 2023 via email

@jaygajera17
Copy link
Owner

there is a conflict in pom.xml file and AdminController.java file in master2 branch.

@NisargPipaliya NisargPipaliya added the Merge Conflict This Branch Has Merge Conflict label Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge Conflict This Branch Has Merge Conflict
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants