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

Add backend support for event groups/roles #3057

merged 4 commits into from Sep 15, 2017


Copy link

@ThiefMaster ThiefMaster commented Sep 6, 2017

No description provided.

['event_group_id'], ['id'],
source_schema='events', referent_schema='events')
ALTER TABLE attachments.attachment_principals DROP CONSTRAINT ck_attachment_principals_valid_enum_type;

This comment has been minimized.


ThiefMaster Sep 6, 2017
Author Member

In case someone wonders why I'm using raw SQL here: It is much less ugly than writing alembic/sqlalchemy statements for it, especially since the CHECK constraint updates are not autogenerated (so like this the dbdiff output can be copied almost as-is, only thing I changed was removing the SET search_path stuff in favor of explicit schemas in table names)

@ThiefMaster ThiefMaster added this to the v2.1 milestone Sep 6, 2017
@ThiefMaster ThiefMaster force-pushed the indico:v2.1-dev branch from b2d75dd to 19567de Sep 11, 2017
@ThiefMaster ThiefMaster force-pushed the ThiefMaster:event-groups branch 2 times, most recently from 0493586 to 19ba872 Sep 11, 2017
@ThiefMaster ThiefMaster force-pushed the indico:v2.1-dev branch from 19567de to 3d34683 Sep 14, 2017
@ThiefMaster ThiefMaster force-pushed the ThiefMaster:event-groups branch from 19ba872 to 0f030a2 Sep 14, 2017
@ThiefMaster ThiefMaster requested a review from pferreir Sep 15, 2017
@ThiefMaster ThiefMaster merged commit 0f030a2 into indico:v2.1-dev Sep 15, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
@ThiefMaster ThiefMaster deleted the ThiefMaster:event-groups branch Sep 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant