You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The CirrusInvitationService (renamed as suggested in the quote below) needs to be based on a new, abstract InvitationService interface. That way, invitation services other than Cirrus can have new services implemented for them, as needed.
Extra credit: What if this was CirrusInvitationService? I guess I was thinking that we could have the class implement an InvitationService interface, and then have it dynamically load the correct module based on what provider is configured. That's probably too fancy at this point, but one of the big hurdles I see to this tool being useful to other institutions is that they either don't have a service like Cirrus or they have another one.
The text was updated successfully, but these errors were encountered:
The
CirrusInvitationService
(renamed as suggested in the quote below) needs to be based on a new, abstractInvitationService
interface. That way, invitation services other than Cirrus can have new services implemented for them, as needed.Originally posted by @ssciolla in #247 (comment)
The text was updated successfully, but these errors were encountered: