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 Develop into master #43

Open
wants to merge 79 commits into
base: master
Choose a base branch
from
Open

Merge Develop into master #43

wants to merge 79 commits into from

Conversation

joelbcastillo
Copy link
Contributor

This PR merges the latest version of the Intranet into develop.

Going forward we plan to have Master always ready to deploy into Production, and use Develop to do work.

johnyu95 and others added 28 commits November 7, 2018 17:18
* develop:
  Added news and updates to dropdown
* develop:
  Removed carousel from git ignore
  Adjusted news and updates spacing
  Added admin description
  Adjust whats new spacing
  Added staff directory to quick links
  Added box shadow to carousel
  Updated gitignore for carousel images
  Updated gitignore for carousel
  Better tag formatting for view post
  Added temp admin description
  Updated admin name for our mission apge
  Updated divison constants
  Added temp description for administration
Feature/IN-118: Local Auth Functionality
Feature/IN-119: NYC.ID Authentication
Feature/IN-114: Orientation Video
* origin/develop:
  Fixed timestamp
  Added orientation video page
  Fixed typo in tags list
Bugfix/IN-125: Intake Form Overwrites Business Owner Information
@joelbcastillo
Copy link
Contributor Author

This pull request introduces 8 alerts when merging 57497bc into 0178a1f - view on LGTM.com

new alerts:

  • 5 for Unused local variable
  • 1 for __eq__ not overridden when adding attributes
  • 1 for Except block handles 'BaseException'
  • 1 for Variable defined multiple times

This project has automated code review enabled, but doesn't use the LGTM GitHub App. Migrate over by installing the app. Read about the benefits of migrating to GitHub Apps in the blog.


Comment posted by LGTM.com

johnyu95 and others added 30 commits April 21, 2022 17:22
added a new table on the database using alembic
created a new ping command intended to be used in a cron job
added and email to be sent to notify application support when a website goes down
…Monitor

* origin/Intranet-Monitor:
  Added 12hr time zone.
… develop

* 'develop' of https://github.com/nycrecords/intranet:
  Uptime monitor code cleanup
  Made correction to image sizing and added warning signs.
  Requiring super user role to access monitor page
  Moved page refresh rate and request timeout to environment variable and fixed modal body text overflow
  Fixed issue with modal not updating on POST and refacting AJAX success code
  Added 12hr time zone.
  Display EST in frontend instead of GMT
  Changing database timestamps to use utc
  Fixed issue where clicking on hyperlink also opened modal
  Linked together monitor frontend and backend
  Added monitor page front end WIP
  Added backend VIEW and AJAX functions to monitor a given set of websites added a new table on the database using alembic created a new ping command intended to be used in a cron job added and email to be sent to notify application support when a website goes down
…cuments

Documents Tab and Quick Links Update
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.

7 participants