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.

Sign up for GitHub

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

Jump to bottom

[Technical] Improve the AA discount group definition #309

Open
celsoMartins opened this issue May 12, 2016 · 0 comments
Open

[Technical] Improve the AA discount group definition #309

celsoMartins opened this issue May 12, 2016 · 0 comments
Labels
low tech debt

Comments

@celsoMartins
Copy link
Member

The current version is using the group name to define it.

def agile_alliance_discount_group
    registration_groups.find_by(name: 'Membros da Agile Alliance')
end

The AA integration is very strong in our domain, so it may make sense to use a boolean field to configure a group that serves to provide discount to AA members.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
low tech debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@celsoMartins