Merge branch 'mrincon/14_7-update-runner-todos' into 'master'

Remove/update outdated to-dos in runner code

See merge request gitlab-org/gitlab!80818

app/assets/javascripts/runner/admin_runners/admin_runners_app.vue

@@ -187,6 +187,7 @@ export default {
       deep: true,
       handler() {
         // TODO Implement back button response using onpopstate
+        // See: https://gitlab.com/gitlab-org/gitlab/-/issues/333804
         updateHistory({
           url: fromSearchToUrl(this.search),
           title: document.title,

app/assets/javascripts/runner/constants.js

@@ -91,8 +91,8 @@ export const ACCESS_LEVEL_REF_PROTECTED = 'REF_PROTECTED';
 // CiRunnerSort
 
 export const CREATED_DESC = 'CREATED_DESC';
-export const CREATED_ASC = 'CREATED_ASC'; // TODO Add this to the API
-export const CONTACTED_DESC = 'CONTACTED_DESC'; // TODO Add this to the API
+export const CREATED_ASC = 'CREATED_ASC';
+export const CONTACTED_DESC = 'CONTACTED_DESC';
 export const CONTACTED_ASC = 'CONTACTED_ASC';
 
 export const DEFAULT_SORT = CREATED_DESC;

app/assets/javascripts/runner/group_runners/group_runners_app.vue

@@ -190,6 +190,7 @@ export default {
       deep: true,
       handler() {
         // TODO Implement back button reponse using onpopstate
+        // See https://gitlab.com/gitlab-org/gitlab/-/issues/333804
         updateHistory({
           url: fromSearchToUrl(this.search),
           title: document.title,