Skip to content

Conversation

@devinmatte
Copy link
Member

New Features:

  • Signature Filter (Total, Upperclassmen, Freshmen)
  • EBoard section gone, Badge for all roles
  • Co-ops properly tracked
  • UI Cleanup

evelyndooley and others added 30 commits September 18, 2018 19:59
Adjust message when congratulating on Slack
I believe this resolves #43
Role Tags for Upperclassmen

@devinmatte asked me to merge this in so we can get it up on develop
Correct Null error in demotion db upgrade
Update to flask-pyoidc version 2.0.0
Google Analytics through config
I personally don't think this is a valuable check
Style and Dependency Updates
Copy link
Collaborator

@mxmeinhold mxmeinhold left a comment

Choose a reason for hiding this comment

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

A few smaller considerations. When I get the chance I'll glance at the web UI again and see if I can break it.

@devinmatte devinmatte requested a review from mxmeinhold August 6, 2019 22:56
Copy link
Member

@JoelEager JoelEager left a comment

Choose a reason for hiding this comment

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

Overall, it looks good to me. Props to everyone who took the time to put work into this over the summer.

@devinmatte devinmatte requested a review from mxmeinhold August 9, 2019 18:35
Copy link
Collaborator

@mxmeinhold mxmeinhold left a comment

Choose a reason for hiding this comment

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

LGTM! 🚢 🙌

@devinmatte devinmatte merged commit 822a75d into master Aug 9, 2019
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.

5 participants