stremio-core icon indicating copy to clipboard operation
stremio-core copied to clipboard

Calendar model

Open tymmesyde opened this issue 1 year ago • 2 comments

tymmesyde avatar Jun 18 '24 10:06 tymmesyde

imo no point of making a pr that does not include all needed dependencies to run. It is untestable. I this case it should include in package.json core-web version that is still in PR without waiting for it to be merged. Preferred solution would be to make this pr only after core and core-web changes are merged so we could avoid testing something without a reason in case it is decided that core/core-web changes should be rewritten or changed in a bigger part.

TRtomasz avatar Sep 18 '24 14:09 TRtomasz

There are a few things in the PR that I would like to change. Especially with handling the Date changes and date changes with offset. I've worked on some of the changes but left it as I had to work on more pressing issues.

Will get back to this.

elpiel avatar Sep 19 '24 08:09 elpiel

I will still test the stremio-web for the UI implementation and general workings of the logic but there are a few things that need refactoring from the model. Logic looks good so far though.

elpiel avatar Oct 25 '24 08:10 elpiel