0us / IE303911-Klubbhuset

Android App m/JavaEE - ID303911 Mobile og distribuerte applikasjoner prosjektoppgave
2 stars 0 forks source link

Integrated has paid function #94

Closed spiftire closed 4 years ago

spiftire commented 4 years ago

Basic logic. not tested. NEEDS TESTING

spiftire commented 4 years ago

closes #93

spiftire commented 4 years ago

Bitte liten typo 😅

fre. 22. nov. 2019, 19:52 skrev Markus Randa notifications@github.com:

@markusranda requested changes on this pull request.

In server/src/main/java/no/ntnu/klubbhuset/service/AdminOrganizationService.java https://github.com/0us/IE303911-Klubbhuset/pull/94#discussion_r349743412 :

@@ -75,4 +76,30 @@ public boolean isAdminOfOrganization(Long organizationId) {

     return result;
 }

+

  • @PermitAll
  • public Response harMemberPaid(Long organizationId, User user) {

"har"memberPaid

Should we really mix Norwegian and English in our naming scheme?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/0us/IE303911-Klubbhuset/pull/94?email_source=notifications&email_token=AKQ5EMAKV6ZEBOZ7BRQNEYDQVAS7XA5CNFSM4JPY2NBKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCMWWJZA#pullrequestreview-321742052, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKQ5EMBQKOBMMWUW6HYMFTLQVAS7XANCNFSM4JPY2NBA .