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 "challenges-dev" into "gssoc20-dev" #59

Merged
merged 12 commits into from
Apr 30, 2020
Merged

Conversation

eshaan7
Copy link
Member

@eshaan7 eshaan7 commented Apr 30, 2020

New features:

  • Multiple boxes support (closes issue Support for n number of boxes #17)
  • Challenges page (closes issue Challenges page similar to machines #56). Along with tag and category models for organization.
  • CTF settings and websites are now stored in database and can easily be changed from the new /setup view. (Closes 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.

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 (closes issues 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 30, 2020

This pull request introduces 5 alerts when merging 0f5157b into 1405748 - view on LGTM.com

new alerts:

  • 5 for Cyclic import

@lgtm-com
Copy link

lgtm-com bot commented Apr 30, 2020

This pull request introduces 3 alerts when merging 5cf7d97 into 1405748 - view on LGTM.com

new alerts:

  • 3 for Wrong name for an argument in a call

…n for model representation and TimeMixin for timestamp fields. dockerfile updated
@eshaan7 eshaan7 merged commit 5bfa08b into gssoc20-dev Apr 30, 2020
@eshaan7 eshaan7 deleted the challenges-dev branch April 30, 2020 20:27
@lgtm-com
Copy link

lgtm-com bot commented Apr 30, 2020

This pull request introduces 2 alerts when merging c77dcfa into 1405748 - view on LGTM.com

new alerts:

  • 2 for Wrong name for an argument in a call

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