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

merge caching-dev into gssoc20-dev #57

Closed
wants to merge 10 commits into from
Closed

Conversation

eshaan7
Copy link
Member

@eshaan7 eshaan7 commented Apr 28, 2020

New features:

  • Multiple boxes support (Issue Support for n number of boxes #17)
  • CTF settings and websites are now stored in database and can easily be changed from the new /setup view. (Issue GUI control of config.py CTF settings #55 )
  • WTForm class for Machine for editing/creating new machines.
  • Implemented thorough and efficient caching with use of "Flask-Caching" and "Redis".
  • Now uses flask factory app method so the various flask extensions can be well-managed.
  • UserMachine model.

Changes:

  • Define redis service in docker-compose.yml
  • Redis add-on has been added to app.json.
  • ADMIN_EMAIL, and mail related env variables added in app.json.
  • Flag submission is now in a modal.
  • DB models are now in their respective folders in models.py
  • Optimized scoreboard's detail fetching
  • Features such as searching and exporting and new dark theme for Admin Controls (Issue Admin Controls, Settings #1, "Admin Controls" - Dark theme #16, Improving the admin controls interface #36 )
  • Some utility functions such as @admin_only decorator.
  • macros have been defined in templates for reusability.

@lgtm-com
Copy link

lgtm-com bot commented Apr 28, 2020

This pull request introduces 1 alert when merging 5a82bbb into 1405748 - view on LGTM.com

new alerts:

  • 1 for 'break' or 'return' statement in finally

@lgtm-com
Copy link

lgtm-com bot commented Apr 28, 2020

This pull request introduces 1 alert when merging f55ad31 into 1405748 - view on LGTM.com

new alerts:

  • 1 for 'break' or 'return' statement in finally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant