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

[NyaaTorrentsBridge] Add published date #3762

Closed
wants to merge 1 commit into from
Closed

[NyaaTorrentsBridge] Add published date #3762

wants to merge 1 commit into from

Conversation

Jisagi
Copy link
Contributor

@Jisagi Jisagi commented Oct 15, 2023

This PR adds the pubDate again. Since it was called date_modified before, I just named it as it was before. Before it was at the top level of the result item, but it now is added to the _rssbridge custom fields.

@github-actions
Copy link

Pull request artifacts

Bridge Context Status
NyaaTorrents 1 untitled (current) ✔️
NyaaTorrents 1 untitled (pr) ✔️

last change: Sunday 2023-10-15 22:25:10

@dvikan
Copy link
Contributor

dvikan commented Oct 15, 2023

i cant find date_modified in its history.

anyways i think this should be timestamp

@dvikan
Copy link
Contributor

dvikan commented Oct 15, 2023

ah i see now that date_modified is what JsonFormat uses to render the timestamp.

still my prior comment is good

@dvikan
Copy link
Contributor

dvikan commented Oct 16, 2023

fixed in #3759

@dvikan dvikan closed this Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants