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

Add new source: PhenixScans #457

Merged
merged 6 commits into from
Sep 10, 2023
Merged

Conversation

MozkaGit
Copy link
Contributor

@MozkaGit MozkaGit commented Sep 9, 2023

Checklist:

  • Updated source's version for individual source changes
  • Updated all sources' versions for template changes
  • Set appropriate nsfw value
  • Did not change id even if a source's name or language were changed
  • Tested the modifications by running it on the simulator or a test device

Changes:

PhenixScans Screenshots:

Source Version Browse In-app Safari
source_version source_browse source_safari
Chapters list Chapter pages Search
source_chapters_list source_chapter source_search

Thank you in advance for the review.

@MozkaGit MozkaGit changed the title Phenix scans Add new source Phenix scans and new icon for Towerofelectrica source Sep 9, 2023
@MozkaGit MozkaGit changed the title Add new source Phenix scans and new icon for Towerofelectrica source Add new source PhenixScans and new icon for Towerofelectrica source Sep 9, 2023
@Skittyblock
Copy link
Owner

Skittyblock commented Sep 10, 2023

prs for different things should remain separate, you should reopen your towerofelectrica pr and you should remove that change from this pr

@MozkaGit MozkaGit changed the title Add new source PhenixScans and new icon for Towerofelectrica source Add new source: PhenixScans Sep 10, 2023
@MozkaGit
Copy link
Contributor Author

Hello, the requested changes have been implemented.

@Skittyblock Skittyblock merged commit 4065457 into Skittyblock:main Sep 10, 2023
2 checks passed
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