Skip to content

Conversation

@abhinavk96
Copy link
Contributor

No description provided.

prateekj117 and others added 30 commits July 12, 2019 15:35
* Restrict unverified user from making free orders.

* Check ticket-type rather than payment-mode

* No need to revert the old PR

* Check if order contains all free tickets.

* Required changes
* add the resend email route

* resend emails using ticket route

* handle errors with UnprocessableEntry

* modify order to user key
fix: Add citext extension in migration if not exists (#6188)
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.0...0.34.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
* handle padding error

* catch specific base64 error
…6217)

Updates the requirements on [apscheduler](https://github.com/agronholm/apscheduler) to permit the latest version.
- [Release notes](https://github.com/agronholm/apscheduler/releases)
- [Changelog](https://github.com/agronholm/apscheduler/blob/master/docs/versionhistory.rst)
- [Commits](agronholm/apscheduler@v3.6.0...v3.6.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
__sudo: required__ no longer is.  The __sudo__ command is now always available in Travis CI and there is no way to turn it off.
> if data['state'] is not 'draft' or not 'pending':
The second half of this if statement is __always__ False because __python -c "print(not 'pending')"__  # --> False

The correct ways to write this line are:
```python
if data['state'] != 'draft' or data['state'] != 'pending':
# or better yet...
if data['state'] not in ('draft', 'pending'):
```
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>
changed the tag from `Yes` to `-`

closes #6185
fixes hound issue

adds an event field to the function
Correct usage of func

Add missing SMTP parameter
self.assertFalse(represents_int('test'))


def test_string_empty(self):

Choose a reason for hiding this comment

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

too many blank lines (2)

EventSubTopicRelationshipRequired, EventSubTopicRelationshipOptional
from app.api.event_topics import EventTopicList, EventTopicDetail, EventTopicRelationship
from app.api.event_types import EventTypeList, EventTypeDetail, EventTypeRelationship
from app.api.events import EventList, EventDetail, EventRelationship, EventCopyResource

Choose a reason for hiding this comment

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

'app.api.events.EventCopyResource' imported but unused

@codecov
Copy link

codecov bot commented Jul 30, 2019

Codecov Report

Merging #6257 into master will decrease coverage by 0.28%.
The diff coverage is 56.01%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6257      +/-   ##
==========================================
- Coverage   66.13%   65.84%   -0.29%     
==========================================
  Files         288      286       -2     
  Lines       14479    14501      +22     
==========================================
- Hits         9575     9548      -27     
- Misses       4904     4953      +49
Impacted Files Coverage Δ
app/api/helpers/ticketing.py 16.21% <ø> (-0.75%) ⬇️
app/api/schema/events.py 91.72% <ø> (+1.79%) ⬆️
app/api/helpers/system_mails.py 100% <ø> (ø) ⬆️
app/api/helpers/mail.py 29.29% <ø> (ø) ⬆️
app/api/schema/sessions.py 86.84% <0%> (ø) ⬆️
app/api/role_invites.py 42.98% <0%> (-1.98%) ⬇️
app/api/helpers/scheduled_jobs.py 21.42% <0%> (-0.18%) ⬇️
app/api/__init__.py 100% <100%> (ø) ⬆️
app/api/uploads.py 30.55% <100%> (ø) ⬆️
tests/all/integration/api/helpers/test_jwt.py 96.96% <100%> (ø) ⬆️
... and 34 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 d1ca881...78fb877. Read the comment docs.

@iamareebjamal iamareebjamal changed the title Merge dev into master for next release chore: Release v1.4.0 Jul 30, 2019
@auto-label auto-label bot added the chore label Jul 30, 2019
@iamareebjamal iamareebjamal merged commit 5163c4c into master Jul 30, 2019
iamareebjamal added a commit that referenced this pull request Jul 30, 2019
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.