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

feat: [#303] store in the database the torrent fields creation_date c… #315

Conversation

mario-nt
Copy link
Contributor

Resolves #303.

@mario-nt mario-nt temporarily deployed to coverage September 26, 2023 13:38 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 26, 2023 13:38 — with GitHub Actions Inactive
@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2023

Codecov Report

Merging #315 (7d9e887) into develop (aabd63c) will decrease coverage by 0.13%.
The diff coverage is 23.07%.

@@             Coverage Diff             @@
##           develop     #315      +/-   ##
===========================================
- Coverage    42.47%   42.34%   -0.13%     
===========================================
  Files           79       79              
  Lines         4801     4822      +21     
===========================================
+ Hits          2039     2042       +3     
- Misses        2762     2780      +18     
Files Coverage Δ
src/services/torrent_file.rs 100.00% <100.00%> (ø)
src/databases/mysql.rs 0.00% <0.00%> (ø)
src/databases/sqlite.rs 23.01% <0.00%> (-0.12%) ⬇️
src/models/torrent.rs 45.94% <0.00%> (-4.06%) ⬇️
src/models/response.rs 0.00% <0.00%> (ø)
src/models/torrent_file.rs 69.86% <0.00%> (-0.73%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 1c9e212 to 07cfb86 Compare September 27, 2023 15:15
@mario-nt mario-nt temporarily deployed to coverage September 27, 2023 15:15 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 27, 2023 15:15 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 07cfb86 to c302735 Compare September 27, 2023 22:02
@mario-nt mario-nt temporarily deployed to coverage September 27, 2023 22:02 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 27, 2023 22:02 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 28, 2023 12:10 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 28, 2023 12:11 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 28, 2023 17:24 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 28, 2023 17:25 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 28, 2023 22:28 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 28, 2023 22:28 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 10:23 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 10:23 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 4f57273 to cbbfe40 Compare September 29, 2023 10:25
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 10:25 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 10:25 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from cbbfe40 to 0866ae3 Compare September 29, 2023 21:34
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 21:35 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 0866ae3 to 911c387 Compare September 29, 2023 21:41
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 21:41 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage September 29, 2023 21:41 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 911c387 to 078e086 Compare October 1, 2023 10:54
@mario-nt mario-nt temporarily deployed to coverage October 1, 2023 10:54 — with GitHub Actions Inactive
@mario-nt mario-nt temporarily deployed to coverage October 1, 2023 10:55 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 078e086 to bdc9932 Compare October 2, 2023 22:02
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 4993082 to af85b6a Compare October 4, 2023 22:59
@mario-nt mario-nt temporarily deployed to coverage October 4, 2023 22:59 — with GitHub Actions Inactive
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from af85b6a to d6ea4a1 Compare October 20, 2023 16:52
@mario-nt mario-nt temporarily deployed to coverage October 20, 2023 16:52 — with GitHub Actions Inactive
Copy link
Member

@josecelano josecelano left a comment

Choose a reason for hiding this comment

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

It looks good!

@josecelano josecelano added the Needs Rebase Base Branch has Incompatibilities label Oct 27, 2023
@josecelano josecelano force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from d6ea4a1 to 7d9e887 Compare October 27, 2023 15:13
@josecelano josecelano temporarily deployed to coverage October 27, 2023 15:13 — with GitHub Actions Inactive
@josecelano
Copy link
Member

Hi @mario-nt this needs a rebase. I tried to do it with the GitHub UI, but commits are not verified.

@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 7d9e887 to 594a70a Compare October 30, 2023 19:57
@mario-nt mario-nt temporarily deployed to coverage October 30, 2023 19:57 — with GitHub Actions Inactive
@mario-nt
Copy link
Contributor Author

@josecelano Rebase done and tests OK, we need to update the types and API libs before merging to make this PR work. I will update the types lib and then update the dependencies in the API lib.

@josecelano josecelano added Needs Rebase Base Branch has Incompatibilities and removed Needs Rebase Base Branch has Incompatibilities labels Oct 31, 2023
@josecelano
Copy link
Member

@josecelano Rebase done and tests OK, we need to update the types and API libs before merging to make this PR work. I will update the types lib and then update the dependencies in the API lib.

I've released the new types https://www.npmjs.com/package/torrust-index-types-lib/v/1.0.0-alpha.4

MMelchor added 4 commits November 7, 2023 16:44
…_date created_by encoding

API responses now contain the three field in the

 - Torrent details endpoint
 - Torrent list endpoint
…tion_date created_by encoding

Fixed linting errors
…tion date created_by encoding

Fix mysql type for creation_date column
… date created_by encoding

Added bindings to the add torrent query in the mysql implementation for the new added fields
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from 594a70a to e8772f3 Compare November 7, 2023 16:03
…tion date created by encoding

    -Added backticks for mysql so the encoding column name is not interpreted as a reserved keyword
@mario-nt mario-nt force-pushed the 303-store-in-the-database-the-torrent-fields-creation-date-created-by-encoding branch from e8772f3 to c22f661 Compare November 9, 2023 16:19
@mario-nt
Copy link
Contributor Author

mario-nt commented Nov 9, 2023

@josecelano Ready to be merged.

@josecelano josecelano removed the Needs Rebase Base Branch has Incompatibilities label Nov 9, 2023
@josecelano
Copy link
Member

ACK c22f661

@josecelano josecelano merged commit 41c80f3 into torrust:develop Nov 9, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Store in the database the torrent fields: creation_date, created_by, encoding
3 participants