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

Fix MangaFox and MangaHere change their domain #35

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix MangaFox and MangaHere change their domain #35

wants to merge 4 commits into from

Conversation

king37sniper
Copy link

@king37sniper king37sniper commented Dec 11, 2017

Fix Mangafox change their domain


This change is Reviewable

@king37sniper
Copy link
Author

king37sniper commented Dec 11, 2017

@braiam pls review these commits

@monkey8113
Copy link

how come this wasnt merged?

@xietbrix
Copy link

i guess reviewer is just busy and haven't gotten around to reviewing it.

@king37sniper
Copy link
Author

I don't know why but he's online a few days a ago
Link : AllMangasReader-dev/AMR#160 (comment)

@king37sniper king37sniper changed the title Fix Mangafox change their domain Fix MangaFox and MangaHere change their domain Dec 19, 2017
@EduardKhudaiberdin
Copy link

Reviewed 1 of 2 files at r1, 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@EduardKhudaiberdin
Copy link

It would be really nice if someone could accept this pull request. @braiam do you have a minute to do that?

@EduardKhudaiberdin
Copy link

Perhaps @inorichi can check accept this pull request. I believe he should have access.

@king37sniper
Copy link
Author

OMG =))

@Stubbs9410
Copy link

pls check AllMangasReader-dev/AMR#164

@Deus-Rex
Copy link

Perhaps it would be more beneficial to instead ask if the repo owner could give a few trusted contributors the required permissions to accept pull requests themselves. I'm sure that would be more reasonable than letting the project remain unmaintained.

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.

6 participants