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

Refactor content requirements. #34

Open
thyrgle opened this issue Feb 13, 2019 · 1 comment
Open

Refactor content requirements. #34

thyrgle opened this issue Feb 13, 2019 · 1 comment

Comments

@thyrgle
Copy link
Owner

thyrgle commented Feb 13, 2019

Currently, the needlemanwunsh.py content method requires too much knowledge on the part of the programmer trying to create problems, more work should be delegated to the problems.py using a decorator or something of the sort. Specifically:

  1. There is no need to handle post requests.
  2. The templates file should not require the problem programmer to import scripts for problem loading, this should be done for them.
  3. There might be the need for some matrix helpers to load from HTML? This part of the API is still up in the air... (More specifically, the matrices differ vastly in internal structure, is there a clean way to encapsulate all of them?)
@thyrgle
Copy link
Owner Author

thyrgle commented Feb 14, 2019

This is now "done", but I am leaving this open for now as I should probably make the manage_content (pseudo-)decorator cleaner.

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

No branches or pull requests

1 participant