Skip to content

Commit 95f4c5e

Browse files
authored
Merge pull request #618 from guardian/ld/intended-audience-cleanup
Some cleanup for the intended audience feature
2 parents 1df6399 + 7f337bf commit 95f4c5e

5 files changed

Lines changed: 1 addition & 22 deletions

File tree

app/controllers/Api.scala

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -208,17 +208,6 @@ class Api(
208208
}
209209
}
210210

211-
def putStubIntendedAudience(stubId: Long) = {
212-
APIAuthAction.async { request =>
213-
ApiResponseFt[Long](for {
214-
json <- readJsonFromRequestResponse(request.body)
215-
intendedAudience <- extractDataResponse[Option[String]](json)
216-
id <- stubsApi.updateIntendedAudience(stubId, intendedAudience)
217-
} yield id
218-
)
219-
}
220-
}
221-
222211
def putStubStatusByComposerId(composerId: String) = {
223212
APIAuthAction.async { request =>
224213
ApiResponseFt[String](for {

common-lib/src/main/scala/com/gu/workflow/api/StubAPI.scala

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -214,13 +214,6 @@ class StubAPI(
214214
item <- extractDataResponse[Long](json)
215215
} yield item
216216

217-
def updateIntendedAudience(stubId: Long, updateIntendedAudience: Option[String]): ApiResponseFt[Long] =
218-
for {
219-
res <- ApiResponseFt.Async.Right(putRequest(s"stubs/$stubId/intendedAudience", updateIntendedAudience.asJson))
220-
json <- parseBody(res.body)
221-
item <- extractDataResponse[Long](json)
222-
} yield item
223-
224217
def updateContentStatusByComposerId(composerId: String, status: String): ApiResponseFt[String] =
225218
for {
226219
res <- ApiResponseFt.Async.Right(putRequest(s"content/$composerId/status", status.asJson))

common-lib/src/main/scala/models/Stub.scala

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,7 @@ case class ExternalData(
5454
rightsDeveloperCommunity: Option[Boolean] = None,
5555
byline: Option[String] = None,
5656
displayHint: Option[String] = None,
57-
trackingTags: Option[List[String]] = None,
58-
intendedAudience: Option[String] = None) {
57+
trackingTags: Option[List[String]] = None) {
5958
}
6059

6160
object ExternalData {

conf/routes

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ PUT /api/stubs/:stubId/plannedNewspaperPublicationDate controller
5252
PUT /api/stubs/:stubId/rightsReviewed controllers.Api.putStubRightsReviewed(stubId: Long)
5353
PUT /api/stubs/:stubId/commissionedLength controllers.Api.putStubCommissionedLength(stubId: Long)
5454
PUT /api/stubs/:stubId/missingCommissionedLengthReason controllers.Api.putStubMissingCommissionedLengthReason(stubId: Long)
55-
PUT /api/stubs/:stubId/intendedAudience controllers.Api.putStubIntendedAudience(stubId: Long)
5655
DELETE /api/stubs/:stubId controllers.Api.deleteStub(stubId: Long)
5756

5857
GET /api/statuses controllers.Api.statusus

public/lib/model/stub.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ export type Stub = {
3030
prodOffice?: string;
3131
note?: string;
3232
displayHint?: string;
33-
intendedAudience?: string;
3433
};
3534

3635
export type ListElementArticleFormat = {

0 commit comments

Comments
 (0)