Project

General

Profile

Actions

Defect #398

closed

Task #348: Sms notification sender

Sending notification from front end does not work

Added by Peter Štrunc about 7 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Peter Štrunc
Category:
Notification
Target version:
Start date:
05/03/2017
Due date:
% Done:

100%

Estimated time:
Affected versions:
Owner:

Description

Sending sample notification by posting on endpoint /notification ends with Content type 'application/hal+json;charset=UTF-8' not supported exception.

Actions #1

Updated by Peter Štrunc about 7 years ago

  • Assignee changed from Vít Švanda to Peter Štrunc
Actions #2

Updated by Peter Štrunc almost 7 years ago

  • Status changed from New to In Progress

I fixed the issue. I will write some tests to prevent future problems and hand it over for review. Implementation continues in branch psourek/smsNotification

Actions #3

Updated by Peter Štrunc almost 7 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Peter Štrunc to Ondřej Kopr
  • % Done changed from 0 to 80

I fixed the issue and wrote integration test that covers this particular aspect of API so it wont happen again.

Fix is in psourek/smsNotification commit d4396f0

Actions #4

Updated by Radek Tomiška almost 7 years ago

  • Status changed from Needs feedback to In Progress
  • Assignee changed from Ondřej Kopr to Peter Štrunc
  • Target version set to Citrine (7.3.0)
  • % Done changed from 80 to 90

It works, thx. Merge fix to develop please.

Test is nice, only remove pls unused service and imports .)

Actions #5

Updated by Peter Štrunc almost 7 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 90 to 100

unused stuff removed and merged in develop

Actions #6

Updated by Radek Tomiška about 5 years ago

  • Tracker changed from Task to Defect
  • Category changed from 20 to Notification
Actions

Also available in: Atom PDF