Skip to content

Commit b80c9c4

Browse files
authored
Merge pull request #2026 from govuk-forms/assign-cards-to-todo-comments
Assign cards to `TODO:` comments
2 parents 91ab446 + e79199b commit b80c9c4

File tree

3 files changed

+4
-1
lines changed

3 files changed

+4
-1
lines changed

app/views/layouts/base.html.erb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
<link rel="apple-touch-icon" href="<%= vite_asset_path "#{govuk_assets_path}/images/govuk-icon-180.png"%>">
1616
<%# This manifest file and the icons it references are currently being served from the `public` folder. %>
1717
<%# TODO: update the build to either generate the manifest with references to fingerprinted assets, or automatically copy the manifest and its dependencies to `public`. %>
18+
<%# https://trello.com/c/zXttaPrR/3450-update-frontend-build-to-handle-manifestjson-automatically %>
1819
<link rel="manifest" href="/manifest.json">
1920
<meta property="og:image" content="<%= vite_asset_path "#{govuk_assets_path}/images/govuk-opengraph-image.png" %>">
2021

config/application.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ class Application < Rails::Application
8888
config.active_record.enumerate_columns_in_select_statements = true
8989

9090
# TODO: remove this when all sensitive data is encrypted
91+
# https://trello.com/c/MXv9NJuX/3459-stop-supporting-unencrypted-data-in-forms-runner
9192
# See https://guides.rubyonrails.org/active_record_encryption.html#support-for-unencrypted-data
9293
config.active_record.encryption.support_unencrypted_data = true
9394

spec/requests/forms/step_controller_spec.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -871,7 +871,8 @@
871871
end
872872
end
873873

874-
# TODO: Need to add test to check how changing an existing routing answer value would work. Better off as a feature spec which we dont have.
874+
# TODO: Add feature test to check how changing an existing routing answer value would work.
875+
# https://trello.com/c/ioopdSWO/3460-add-a-test-for-changing-an-existing-answer-with-routing
875876
end
876877

877878
context "when page is repeatable" do

0 commit comments

Comments
 (0)