common/app/model/dotcomrendering/DotcomRenderingDataModel.scala (3 lines): - line 57: webPublicationDateDisplay: String, // TODO remove - line 567: .filter(_.published || pageType.isPreview) // TODO lift? - line 649: pageType = pageType, // TODO this info duplicates what is already elsewhere in format? common/app/model/dotcomrendering/pageElements/PageElement.scala (3 lines): - line 35: // TODO dates are being rendered as strings to avoid duplication of the - line 887: // TODO we should quick fail here for these rather than pointlessly go to DCR - line 2018: TODO: common/app/model/dotcomrendering/Trail.scala (3 lines): - line 20: image: Option[String], // TODO: This can be deprecated after 'masterImage' is supported in DCR - line 21: carouselImages: Map[String, Option[String]], // TODO: This can be deprecated after 'masterImage' is supported in DCR - line 82: // TODO: Replace this. common/app/common/jobs.scala (2 lines): - line 49: // TODO delete this function and make easy to understand function(s) - line 53: ) // TODO make a version to take a future common/app/common/RequestLogger.scala (2 lines): - line 27: "Accept-Encoding", // TODO remove if seen after 2021/09/03 - line 66: "resp.Vary" -> r.header.headers.getOrElse("Vary", ""), // TODO remove if seen after 2021/09/03 static/src/javascripts/projects/common/modules/commercial/acquisitions-link-enrichment.js (2 lines): - line 76: // TODO: remove this when only https://github.com/guardian/acquisition-iframe-tracking - line 77: // TODO: is being used for acquisition iframe tracking static/src/stylesheets/module/commercial/_creatives.scss (2 lines): - line 37: /*TODO consider using solution from https://www.youtube.com/watch?v=QU1JAW5LRKU*/ - line 200: /*TODO consider using solution from https://www.youtube.com/watch?v=QU1JAW5LRKU*/ admin/app/controllers/admin/R2PressController.scala (2 lines): - line 64: // TODO: other validation? - line 113: // TODO: other validation? static/src/javascripts/projects/common/modules/atoms/youtube-player.ts (2 lines): - line 100: // TODO: Fix this so we can add poster image. - line 147: * TODO: Use buildAdsConfig from `@guardian/commercial` sport/app/rugby/model/paRugby.scala (2 lines): - line 73: Stage.Group // TODO figure out how to get stage from PA data as not obvious - line 85: case "Extra time" => Status.ExtraTimeFirstHalf // TODO not equivalent static/src/stylesheets/module/_popup.scss (2 lines): - line 8: // TODO: - line 136: // TODO: Combine parent selector (@at-root) or move context class static/src/javascripts/projects/common/modules/commercial/user-features.ts (2 lines): - line 42: // TODO: isn’t this duplicated from commercial features? - line 83: * TODO: check that this validation is accurate onward/app/views/fragments/videosInSeries.scala.html (2 lines): - line 21: - line 22: sport/app/football/controllers/MoreOnMatchController.scala (1 line): - line 23: // TODO import java.time.LocalDate and do not import DateHelpers. common/app/model/Tag.scala (1 line): - line 116: * TODO: figure out if this is actually being used. If so, refactor it. common/app/model/trails.scala (1 line): - line 92: /** TODO - this should be set in the Facia tool */ common/app/views/fragments/items/elements/facia_cards/kicker.scala.html (1 line): - line 6: @* TODO: this is only used by dream snaps now (see title.scala.html for the actual kicker branching code for common/app/layout/DateHeadline.scala (1 line): - line 11: // TODO add a month endpoint and then make this non-optional common/app/views/support/JavaScriptPage.scala (1 line): - line 80: // TODO: decide whether the value for `isDev` should be static/src/javascripts/projects/common/modules/discussion/comment-box.js (1 line): - line 289: // TODO: this should be wrapped into a try-catch block identity/app/form/PrivacyMapping.scala (1 line): - line 54: /** FIXME: Once GDPR goes live, clean Mongo DB of old consents, and remove this method. common/app/common/Edition.scala (1 line): - line 149: // TODO - understand RSS static/src/stylesheets/layout/nav/_subnav.scss (1 line): - line 35: // TODO: reset list-style mixin commercial/app/views/fragments/amp/customStyles.scala.html (1 line): - line 7: @* TODO: these two stylesheets should be refactored into SCSS *@ static/src/javascripts/bootstraps/enhanced/main.js (1 line): - line 148: // TODO: consider issues with double-loading bundles in hosted + standalone commercial (@mxdvl 2021-09-30) common/app/layout/FaciaHeaderImageType.scala (1 line): - line 5: /** TODO fancy cut out version */ static/src/stylesheets/layout/nav/_variables.scss (1 line): - line 14: // TODO: replace with pillar height variable static/src/javascripts/boot.js (1 line): - line 154: // TODO improve shady pie to make better use of the slot. common/app/model/meta.scala (1 line): - line 250: // TODO I am sure these should be in shared code, but not aware of any at time common/app/common/configuration.scala (1 line): - line 685: // TODO needs the orElse fallback till we roll out new properties files admin/app/football/model/PrevResult.scala (1 line): - line 21: // TODO: check if this class is used anywhere common/app/navigation/Navigation.scala (1 line): - line 19: // TODO: Shouldn't need iconName. Remove, and just make the first NavLink on mobile static/src/stylesheets/module/facia-garnett/_tag-meta.scss (1 line): - line 1: //TODO This and the corresponding templates in discussion must be refactored to new fc-containers common/app/views/fragments/nav/headerMenu.scala.html (1 line): - line 74: @* TODO: clean up search menu desktop styling *@ common/app/views/support/GetClasses.scala (1 line): - line 120: /** TODO get rid of this when we consolidate 'all' logic with index logic */ static/src/javascripts/bootstraps/enhanced/accessibility.js (1 line): - line 23: // TODO damn me when I decided to implement this page sport/app/rugby/model/rugby.scala (1 line): - line 16: event: Event, // TODO rename to just RugbyEvent (as will soon be PA) common/app/common/TrailsToShowcase.scala (1 line): - line 106: // TODO: figure out how to include the collection(s) common/app/layout/slices/FixedContainers.scala (1 line): - line 8: // TODO: Temporary vals for content until we refactor common/app/model/ShareLinks.scala (1 line): - line 122: // TODO: Use campaign codes common/app/model/VideoPlayer.scala (1 line): - line 19: // TODO make `String` once `path` is available on main media on fronts common/app/model/dotcomrendering/DotcomRenderingSupportTypes.scala (1 line): - line 68: // TODO simplify date fields when DCR is ready common/app/controllers/EmailSignupController.scala (1 line): - line 77: // FIXME: this should go via the identity api client / app common/app/layout/ContentWidths.scala (1 line): - line 196: tablet = Some(700.px), // TODO: Change to 480 when new galleries is merged identity/app/idapiclient/responses/Error.scala (1 line): - line 5: /** FIXME: This seems to be exactly the same as com.gu.identity.model.Error from identity-request library. Is this only common/app/layout/FaciaContainer.scala (1 line): - line 92: * TODO: change Facia Tool to have a dropdown for 'header types', one of which is default, the other CP Scott. applications/app/controllers/GalleryController.scala (1 line): - line 40: Future.successful(RenderOtherStatus(Gone)) // TODO - delete this line after switching to new content api static/src/stylesheets/amp/_content.scss (1 line): - line 223: // TODO: on amp, shouldn't have this. Or, use amp-lightbox static/src/javascripts/projects/commercial/commercial-metrics.ts (1 line): - line 31: // TODO: rename upstream static/src/stylesheets/module/content-garnett/_interactive.scss (1 line): - line 122: //TODO: Temporary hack for live-streams until we discuss a better solution with composer static/src/javascripts/projects/common/modules/charts/doughnut.js (1 line): - line 72: // TODO (jamesgorrie): functionalise common/app/model/Cached.scala (1 line): - line 69: apply(CacheTime(seconds), result, request.headers.get("If-None-Match")) // FIXME could be comma separated common/app/views/fragments/containers/index.scala.html (1 line): - line 12: @* TODO Consolidate this with the master container template to get rid of repetition *@ static/src/javascripts/projects/commercial/modules/hosted/gallery.js (1 line): - line 484: // TODO: If we add `states` to the list of annotations within the class, it is `undefined` in the constructor. Wat? static/src/javascripts/projects/common/modules/experiments/ab-utils.ts (1 line): - line 57: // TODO: enable noUncheckedIndexedAccess in tsconfig common/app/implicits/FaciaContentFrontendHelpers.scala (1 line): - line 25: // TODO: Use the blocks field of CAPI to derive this in a more structured way onward/app/controllers/NavigationController.scala (1 line): - line 56: // TODO: static/src/stylesheets/module/email-signup/_newDesign.scss (1 line): - line 24: /* TODO: Remove these and update iframe class post-AB test */ common/app/views/fragments/email/signup/subscription/emailSignUp.scala.html (1 line): - line 12: @* TODO - remove custom tones? static/src/stylesheets/module/content-garnett/_article-splash.scss (1 line): - line 41: // TODO this should be done in a more intelligent way static/src/javascripts/projects/common/modules/identity/modules/loadEnhancers.js (1 line): - line 4: /* TODO:Improve type checking here */ identity/app/idapiclient/IdApiClient.scala (1 line): - line 275: .withBody(body.getOrElse("")) // FIXME: DELETE should not have a body static/src/stylesheets/module/commercial/paidfor/_paidfor-band.scss (1 line): - line 31: // TODO: switch to focus-visible when browsers support it static/src/javascripts/bootstraps/enhanced/common.js (1 line): - line 2: /* TODO - fix module constructors */ sport/app/football/feed/agent.scala (1 line): - line 111: } // TODO it would be nice to handle expected errors more gracefully (e.g. like 'No data available', which happens in between seasons) static/src/javascripts/projects/common/modules/navigation/search.js (1 line): - line 6: // TODO refactor to singleton common/app/model/Asset.scala (1 line): - line 117: // FIXME: Remove this once the multimedia.guardianapis are available over https