Skip to content

Conversation

@iamareebjamal
Copy link
Member

No description provided.

dependabot-preview bot and others added 30 commits September 10, 2019 13:27
Updates the requirements on [arrow](https://github.com/crsmithdev/arrow) to permit the latest version.
- [Release notes](https://github.com/crsmithdev/arrow/releases)
- [Changelog](https://github.com/crsmithdev/arrow/blob/master/CHANGELOG.md)
- [Commits](arrow-py/arrow@0.14.5...0.15.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [stripe](https://github.com/stripe/stripe-python) to permit the latest version.
- [Release notes](https://github.com/stripe/stripe-python/releases)
- [Changelog](https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md)
- [Commits](stripe/stripe-python@v2.35.0...v2.35.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sqlalchemy-utils](https://github.com/kvesteri/sqlalchemy-utils) to permit the latest version.
- [Release notes](https://github.com/kvesteri/sqlalchemy-utils/releases)
- [Changelog](https://github.com/kvesteri/sqlalchemy-utils/blob/master/CHANGES.rst)
- [Commits](kvesteri/sqlalchemy-utils@0.34.1...0.34.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [arrow](https://github.com/crsmithdev/arrow) to permit the latest version.
- [Release notes](https://github.com/crsmithdev/arrow/releases)
- [Changelog](https://github.com/crsmithdev/arrow/blob/master/CHANGELOG.md)
- [Commits](arrow-py/arrow@0.15.0...0.15.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [stripe](https://github.com/stripe/stripe-python) to permit the latest version.
- [Release notes](https://github.com/stripe/stripe-python/releases)
- [Changelog](https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md)
- [Commits](stripe/stripe-python@v2.35.1...v2.36.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [flask-jwt-extended](https://github.com/vimalloc/flask-jwt-extended) to permit the latest version.
- [Release notes](https://github.com/vimalloc/flask-jwt-extended/releases)
- [Commits](vimalloc/flask-jwt-extended@3.21.0...3.23.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sqlalchemy](https://github.com/sqlalchemy/sqlalchemy) to permit the latest version.
- [Release notes](https://github.com/sqlalchemy/sqlalchemy/releases)
- [Changelog](https://github.com/sqlalchemy/sqlalchemy/blob/master/CHANGES)
- [Commits](https://github.com/sqlalchemy/sqlalchemy/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
* fix: should not return fields related to a deleted field

* Add condition for get_trashed query param

* Remove duplication of code

* Remove duplication of one

* Checking dredd tests

* Add check to see if 'deleted_at' attribute exists.

* Undo changes in hook_main
Updates the requirements on [pycryptodome](https://github.com/Legrandin/pycryptodome) to permit the latest version.
- [Release notes](https://github.com/Legrandin/pycryptodome/releases)
- [Changelog](https://github.com/Legrandin/pycryptodome/blob/master/Changelog.rst)
- [Commits](Legrandin/pycryptodome@v3.8.2...v3.9.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [stripe](https://github.com/stripe/stripe-python) to permit the latest version.
- [Release notes](https://github.com/stripe/stripe-python/releases)
- [Changelog](https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md)
- [Commits](stripe/stripe-python@v2.36.0...v2.36.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sendgrid](https://github.com/sendgrid/sendgrid-python) to permit the latest version.
- [Release notes](https://github.com/sendgrid/sendgrid-python/releases)
- [Changelog](https://github.com/sendgrid/sendgrid-python/blob/master/CHANGELOG.md)
- [Commits](sendgrid/sendgrid-python@v6.0.0...v6.1.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [stripe](https://github.com/stripe/stripe-python) to permit the latest version.
- [Release notes](https://github.com/stripe/stripe-python/releases)
- [Changelog](https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md)
- [Commits](stripe/stripe-python@v2.36.1...v2.36.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [arrow](https://github.com/crsmithdev/arrow) to permit the latest version.
- [Release notes](https://github.com/crsmithdev/arrow/releases)
- [Changelog](https://github.com/crsmithdev/arrow/blob/master/CHANGELOG.rst)
- [Commits](arrow-py/arrow@0.15.1...0.15.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sentry-sdk](https://github.com/getsentry/sentry-python) to permit the latest version.
- [Release notes](https://github.com/getsentry/sentry-python/releases)
- [Changelog](https://github.com/getsentry/sentry-python/blob/master/CHANGES.md)
- [Commits](getsentry/sentry-python@0.11.0...0.12.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [pylint](https://github.com/PyCQA/pylint) to permit the latest version.
- [Release notes](https://github.com/PyCQA/pylint/releases)
- [Changelog](https://github.com/PyCQA/pylint/blob/master/ChangeLog)
- [Commits](pylint-dev/pylint@pylint-2.3.0...pylint-2.4.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
* feat: Converted TicketHolder Model Def to dataclass

* added unsafe_hash in decorator params

* Changed Python version in Travis config file

* Update .travis.yml
Updates the requirements on [stripe](https://github.com/stripe/stripe-python) to permit the latest version.
- [Release notes](https://github.com/stripe/stripe-python/releases)
- [Changelog](https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md)
- [Commits](stripe/stripe-python@v2.36.2...v2.37.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
chore: Merge master into dev
Updates the requirements on [pillow](https://github.com/python-pillow/Pillow) to permit the latest version.
- [Release notes](https://github.com/python-pillow/Pillow/releases)
- [Changelog](https://github.com/python-pillow/Pillow/blob/master/CHANGES.rst)
- [Commits](python-pillow/Pillow@6.1.0...6.2.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
iamareebjamal and others added 23 commits October 2, 2019 14:07
Updates the requirements on [flask-limiter](https://github.com/alisaifee/flask-limiter) to permit the latest version.
- [Release notes](https://github.com/alisaifee/flask-limiter/releases)
- [Changelog](https://github.com/alisaifee/flask-limiter/blob/master/HISTORY.rst)
- [Commits](alisaifee/flask-limiter@1.0.1...1.1.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sqlalchemy](https://github.com/sqlalchemy/sqlalchemy) to permit the latest version.
- [Release notes](https://github.com/sqlalchemy/sqlalchemy/releases)
- [Changelog](https://github.com/sqlalchemy/sqlalchemy/blob/master/CHANGES)
- [Commits](https://github.com/sqlalchemy/sqlalchemy/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [stripe](https://github.com/stripe/stripe-python) to permit the latest version.
- [Release notes](https://github.com/stripe/stripe-python/releases)
- [Changelog](https://github.com/stripe/stripe-python/blob/master/CHANGELOG.md)
- [Commits](stripe/stripe-python@v2.37.1...v2.37.2)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sqlalchemy](https://github.com/sqlalchemy/sqlalchemy) to permit the latest version.
- [Release notes](https://github.com/sqlalchemy/sqlalchemy/releases)
- [Changelog](https://github.com/sqlalchemy/sqlalchemy/blob/master/CHANGES)
- [Commits](https://github.com/sqlalchemy/sqlalchemy/commits)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Updates the requirements on [sentry-sdk](https://github.com/getsentry/sentry-python) to permit the latest version.
- [Release notes](https://github.com/getsentry/sentry-python/releases)
- [Changelog](https://github.com/getsentry/sentry-python/blob/master/CHANGES.md)
- [Commits](getsentry/sentry-python@0.12.0...0.13.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
* Add rest endpoint for server version

* Fix error message

* Fix the hound issues

* Change route to '/info' and modify json to include timestamp

* Remove the class and define the method only for /info route

* move import and register blueprint to end

* register blueprint to end for info route

* put register blueprint for info_route back

* Move the register blueprints to original place

* Update __init__.py
@auto-label auto-label bot added the chore label Oct 19, 2019
with stash['app'].app_context():
event = EventFactoryBasic()
db.session.add(event)
RoleFactory(name=OWNER) # TODO: Change to get_or_create in event after_created

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at least two spaces before inline comment

from flask import Flask
from app.models import db
from populate_db import populate_without_print
from app.models.user import OWNER

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

module level import not at top of file

sys.path.insert(1, path.abspath(path.join(__file__, "../..")))

from flask_migrate import Migrate, stamp
from flask_migrate import Migrate

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

module level import not at top of file

with app.test_request_context():
result = check_migrations().split(',')
self.assertEqual(result[0], 'success')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

blank line contains whitespace


# https://docs.sqlalchemy.org/en/13/faq/connections.html#how-do-i-use-engines-connections-sessions-with-python-multiprocessing-or-os-fork
def add_engine_pidguard(engine):
from sqlalchemy import event, exc # placed here for import conflict resolution

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

at least two spaces before inline comment

record = self.session.query(model).filter(getattr(model, column_name) == value)
if request.args.get('get_trashed') != 'true':
if hasattr(model, 'deleted_at'):
record = record.filter(model.deleted_at == None)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comparison to None should be 'if cond is None:'

@codecov
Copy link

codecov bot commented Oct 19, 2019

Codecov Report

Merging #6541 into master will increase coverage by 0.57%.
The diff coverage is 91.62%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6541      +/-   ##
==========================================
+ Coverage   64.55%   65.12%   +0.57%     
==========================================
  Files         292      296       +4     
  Lines       15158    15292     +134     
==========================================
+ Hits         9785     9959     +174     
+ Misses       5373     5333      -40
Impacted Files Coverage Δ
tests/all/integration/setup_database.py 96.15% <ø> (-0.28%) ⬇️
tests/all/integration/test_migrations.py 100% <100%> (ø) ⬆️
app/api/helpers/ticketing.py 16.8% <100%> (ø) ⬆️
...all/integration/api/helpers/test_scheduled_jobs.py 100% <100%> (ø)
app/factories/session.py 100% <100%> (ø) ⬆️
app/models/speaker.py 94.66% <100%> (ø) ⬆️
config.py 88% <100%> (+0.16%) ⬆️
app/api/helpers/pentabarfxml.py 100% <100%> (+40.47%) ⬆️
app/api/helpers/scheduled_jobs.py 23.43% <100%> (+2.5%) ⬆️
app/factories/microlocation.py 100% <100%> (ø) ⬆️
... and 22 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c1a6a0e...37c7097. Read the comment docs.

@abhinavk96 abhinavk96 temporarily deployed to open-event-server-dev October 19, 2019 17:32 Inactive
@iamareebjamal iamareebjamal temporarily deployed to open-event-api-dev October 19, 2019 17:32 Inactive
@iamareebjamal iamareebjamal merged commit ca99273 into master Oct 19, 2019
iamareebjamal added a commit that referenced this pull request Oct 19, 2019
chore: Release 1.7.0 to master (#6541)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.