Skip to content

Fix parsing datetime#1

Merged
OmkarDabade merged 1 commit into
OmkarDabade:masterfrom
piotruela:fix/parsing-datetime
Jun 19, 2024
Merged

Fix parsing datetime#1
OmkarDabade merged 1 commit into
OmkarDabade:masterfrom
piotruela:fix/parsing-datetime

Conversation

@piotruela

Copy link
Copy Markdown

I reused a code from a pull request to the original webfeed package. Original package seems to be discontinued :/

@And96

And96 commented Aug 27, 2023

Copy link
Copy Markdown

Yes. Absolutly. Parsing date has always been wrong even on original package.
It work only on 3/4 of sites

I think webfeed_revised (0.7.2) should have fixed that but I didnt tested all date formats.

I forked original "webfeed" (0.7.0) repo and rewritten date format manually. This work perfectly
https://github.com/And96/webfeed_simplified

@OmkarDabade OmkarDabade merged commit 35e664f into OmkarDabade:master Jun 19, 2024
@OmkarDabade

Copy link
Copy Markdown
Owner

PR merged. Thank you @piotruela

@And96

And96 commented Jul 4, 2024

Copy link
Copy Markdown

There are also others timezone:

//actually

const timeZoneAbbreviations = {
  'EET': 2 * 60,
  'CET': 1 * 60,
  'GMT': 0,
  'AST': -4 * 60,
  'EST': -5 * 60,
  'EDT': -4 * 60,
  'CST': -6 * 60,
  'CDT': -5 * 60,
  'MST': -7 * 60,
  'MDT': -6 * 60,
  'PST': -8 * 60,
  'PDT': -7 * 60,
};

here should be the full list:

("PDT","-0700")
("PST","-0800")
("PT","-0800")
("MST","-0700")
("MDT","-0600")
("MT","-0700")
("CST","-0600")
("CDT","-0500")
("CT","-0600")
("EST","-0500")
("EDT","-0400")
("ET","-0500")
("AST","-0400")
("GMT","-0000")
("CET","+0100")
("EET","+0200")

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.

3 participants