-
Notifications
You must be signed in to change notification settings - Fork 69
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
Komga implementation #943
Merged
Merged
Komga implementation #943
Changes from 52 commits
Commits
Show all changes
70 commits
Select commit
Hold shift + click to select a range
4f15900
komga implementation
Jacob-Tate b5b39cd
Updated Komga integration to use SSE
Jacob-Tate 238b146
Merge branch 'IgnisDa:main' into komga_impl
Jacob-Tate 0e1c957
Finalizing implementation
Jacob-Tate 83105fe
Added Komga Documentation
Jacob-Tate 9fc4687
Placed Cargo.toml in Alphabetic Order
Jacob-Tate 6bd89bd
Merge branch 'IgnisDa:main' into komga_impl
Jacob-Tate 3e1f70a
Changed Komga into a Yank Integration.
Jacob-Tate 86e2c5c
Removed provider specifics as it wasnt really needed.
Jacob-Tate b43df5b
Reverted order
Jacob-Tate 834f3ff
Updated komga documentation
Jacob-Tate fb43a33
Fixed a bug where some manga have imported with the incorrect number
Jacob-Tate 0b38409
docs: address formatting issues
IgnisDa 6ab65c4
build(backend): pin `eventsource-stream` version
IgnisDa 5aab4c8
chore(backend): format files
IgnisDa 828ef34
refactor(backend): move integration to dedicated folder
IgnisDa a7a2b55
Fixed a bug where some tachiyomi clients report in random order.
Jacob-Tate 8920c8a
Removed provider specifics from UpdateUserInntegrationInput
Jacob-Tate 647cf47
Merge remote-tracking branch 'refs/remotes/origin/main' into komga_impl
Jacob-Tate ff38c8c
Fixed order of includes to match original.
Jacob-Tate ebd9be3
Fixed order of imports for komga.rs
Jacob-Tate 54cd048
chore(backend): run formatter
IgnisDa 94ba3f6
chore(backend): some housekeeping
IgnisDa 57bff2d
fix(backend): type checks
IgnisDa ba0980d
chore(backend): add newline
IgnisDa 72157b4
feat(backend): yank integrations data right on application startup
IgnisDa ecaf309
chore(backend): add a debug statement
IgnisDa 30c9b78
Removed Option from find_provider_and_id
Jacob-Tate bc28164
Merge branch 'main' into komga_impl
IgnisDa 302cb13
Run CI Empty Commit
Jacob-Tate afd70a8
chore: empty commit
IgnisDa c4070e3
ci: additional check for maintainer
IgnisDa 5112364
merge origin
Jacob-Tate 3d21694
Komga Implementation
Jacob-Tate 703dc30
Merge branch 'refs/heads/komga' into komga_impl
Jacob-Tate 1b02fc5
Resolve merge errors
Jacob-Tate 4469588
Merge branch 'komga_impl' of https://github.com/jacob-tate/ryot into …
IgnisDa 5c9c4f0
ci: revert ci changes
IgnisDa 01f8d7f
chore(integrations): import structure
IgnisDa f2397b1
build(integrations): remove uneeded package
IgnisDa cc8178d
ci: change computation of image names
IgnisDa 36bd0e3
ci: hardcode docker username
IgnisDa 3180b9b
ci: enable hardmode
IgnisDa 9b37a40
ci: quote names
IgnisDa 6f6c2c9
ci: no heredoc fuckery
IgnisDa e3e0905
ci: change name of image
IgnisDa dd54876
ci: remove docker for forks
IgnisDa 12cae01
ci: update condition
IgnisDa eccf2c4
ci: logging
IgnisDa 6debb41
ci: use gh actions script
IgnisDa a9d23bb
ci: print all outputs
IgnisDa ea960dc
ci: use correct image names
IgnisDa c41b365
ci: use correct owner name
IgnisDa ba08f71
Upgraded to a trait model
Jacob-Tate fe023cd
Migrated jellyfin to trait
Jacob-Tate 2834ac2
Fixed visibility of function
Jacob-Tate 0f528a2
Emby Integration Migration
Jacob-Tate 110d25c
Plex Integration Migration
Jacob-Tate bc5dfcc
Audiobookshelf Integration Migration (partial)
Jacob-Tate 45db825
Kodi Integration Migration
Jacob-Tate 5f131ed
Sonarr Integration Migration
Jacob-Tate be9c608
Radarr Integration Migration
Jacob-Tate 3fc61c0
Ran formatter
Jacob-Tate 8ecc8a6
chore: run rust formatter
IgnisDa 79c6c05
refactor(integrations): some import changes
IgnisDa 6715016
Finished implementing Audiobookshelf
Jacob-Tate 3eebdb4
Undid accidental change
Jacob-Tate 2394396
Merge branch 'IgnisDa:main' into komga_impl
Jacob-Tate ec018ed
chore(backend): apply formatter
IgnisDa c1eba11
Merge branch 'main' into komga_impl
IgnisDa File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -454,6 +454,7 @@ pub enum IntegrationProvider { | |
Kodi, | ||
Radarr, | ||
Sonarr, | ||
Komga, | ||
} | ||
|
||
#[derive( | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vnghia I have been trying to get this working for some time, but it seems that there is no way to publish a docker image from a PR (from a fork) since the
GITHUB_TOKEN
granted to the workflow only has read permissions (despite specifying write explicitly in the action).Do you know a solution to this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So the current flow never works for PRs. Only for PRs from a branch in the same repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let me think about it. you can use
env
instead ofsecrets
but it will print your env in the logThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But to actually inject the secrets into env, i would still need access to secrets. Which are not available in PRs from forks.