Skip to content

Comments

Feature/issue 184/create sqlite database to store user information#247

Merged
Siyuan106 merged 142 commits intodevfrom
feature/issue-184/create-sqlite-database-to-store-user-information
Apr 3, 2025
Merged

Feature/issue 184/create sqlite database to store user information#247
Siyuan106 merged 142 commits intodevfrom
feature/issue-184/create-sqlite-database-to-store-user-information

Conversation

@Liuwf4319
Copy link
Contributor

Issue(s):

#184

Type of change: (choose required ones)

  • Bug fix
  • New feature
  • Refactor/Optimization

Description:

Added database, changed some backend methods to fit.

Additional context:

N/A

Testing instructions:

do "mvn clean test". Confirm that all tests pass
"mvn spring-boot:run" to view database interactions

Siyuan106 and others added 30 commits February 9, 2025 20:04
Add dependency for SQLite
Add code to implement db.
Change the local data to the database
Change the local data to the database

Co-authored-by: Siyuan106 <gr23994@bristol.ac.uk>
Co-authored-by: Siyuan106 <gr23994@bristol.ac.uk>
forget sendMessage
Add dependencies for sqlite.
Using SQLite database instead of local storage.
vladislav-k1
vladislav-k1 previously approved these changes Apr 3, 2025
Copy link
Contributor

@vladislav-k1 vladislav-k1 left a comment

Choose a reason for hiding this comment

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

The database works, even after restarting the backend. Good job.

vladislav-k1
vladislav-k1 previously approved these changes Apr 3, 2025
Copy link
Contributor

@vladislav-k1 vladislav-k1 left a comment

Choose a reason for hiding this comment

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

The database works, even after restarting the backend. Good job.

Copy link
Contributor

@Siyuan106 Siyuan106 left a comment

Choose a reason for hiding this comment

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

The database worked well!

Copy link
Contributor

@RainBOY-ZZX RainBOY-ZZX left a comment

Choose a reason for hiding this comment

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

LGTM

@Siyuan106 Siyuan106 merged commit 1ce97c1 into dev Apr 3, 2025
2 checks passed
@Siyuan106 Siyuan106 deleted the feature/issue-184/create-sqlite-database-to-store-user-information branch April 3, 2025 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Issue relates to backend code. Final Issue relates to Final release. Frontend Issue relates to frontend code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants