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

Move Page-specific Code Into ./Pages Directory #14

Open
Toxocious opened this issue Apr 23, 2022 · 0 comments
Open

Move Page-specific Code Into ./Pages Directory #14

Toxocious opened this issue Apr 23, 2022 · 0 comments
Labels
refactor Desirable refactoring of the feature/script in the issue

Comments

@Toxocious
Copy link
Owner

Description

To further organize the code-base, code regarding specific pages should be moved into the ./pages directory, similar to how all of the code regarding the Pokemon Center lies in the ./pages/pokemon_center directory.

Doing this would also benefit from some refactoring of the code for each page that would be moved to the ./pages directory, where the new code would be organized as such:

./pages
|---- pokemon_center  ## Feature Name 
    |---- ajax        ## AJAX content, sent as JSON
        |---- inventory.php
        |---- moves.php
        |---- nickname.php
        |---- release.php
        |---- roster.php
    |---- functions   ## PHP functions
        |---- inventory.php
        |---- moves.php
        |---- nickname.php
        |---- release.php
        |---- roster.php
    |---- js          ## JS for handling client input
        |---- ajax_functions.js
        |---- inventory.js
        |---- moves.js
        |---- nickname.js
        |---- release.js
        |---- roster.js
    |---- pages       ## Base page templating
        |---- inventory.php
        |---- moves.php
        |---- nickname.php
        |---- release.php
        |---- roster.php

Pages To Move And Refactor

  • Clans
  • Direct Messages
  • Evolution Center
  • Profile
  • Rankings
  • Shop
  • Trade Center
@Toxocious Toxocious added the refactor Desirable refactoring of the feature/script in the issue label Apr 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Desirable refactoring of the feature/script in the issue
Projects
Status: Todo
Development

No branches or pull requests

1 participant