Use AjaxCache in Droplab Ajax plugin

What does this MR do?

Use AjaxCache instead of Droplabs internal cache for dropdown data.

Why was this MR needed?

Previously, duplicate requests were made to the same URL from dropdown and non-dropdown code.

What are the relevant issue numbers?

fixes #31897 (closed)

LGTM!

changed milestone to %9.2

mentioned in merge request !11170 (merged)

@jschatz1 Can you please review? This solves the duplicate request you were experiencing.

unmarked as a Work In Progress

assigned to @jschatz1

mentioned in commit b8f27513

added 23 commits

• bd8db1f8...387b4410 - 22 commits from branch master
• b8f27513 - Use AjaxCache in Droplab Ajax plugin (!11171 (merged))

Compare with previous version

You have failing tests. Wonder if it is master that is failing.

I think https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/11222 attempts to solve the failing test.

Yeah, sorry about that. I hoped master would turn green yesterday.

Assigning to me until tests pass again.

marked as a Work In Progress

assigned to @winh

mentioned in commit 1111bec6

added 116 commits

• b8f27513...1daa133c - 115 commits from branch master
• 1111bec6 - Use AjaxCache in Droplab Ajax plugin (!11171 (merged))

Compare with previous version

@jschatz1 Rebased on master and tests are passing now.

ee_compat_check failed, so I'll create an EE merge request as soon as this is reviewed.

unmarked as a Work In Progress

assigned to @jschatz1

approved this merge request

@ClemMakesApps you wanna look at this?

assigned to @ClemMakesApps

Just one comment and everything else LGTM. Thanks for this @winh

assigned to @winh

mentioned in commit 382995c2

resolved all discussions

added 1 commit

• 382995c2 - Use AjaxCache in Droplab Ajax plugin (!11171 (merged))

Compare with previous version

assigned to @ClemMakesApps

@winh looks like this requires @lbennett approval

assigned to @lbennett

Did we decide to make this %9.3 ?

approved this merge request

Thanks @winh and @ClemMakesApps.

@ClemMakesApps IIRC this was an ideal piece missing from a deliverable so should be picked. Sorry RMs! :(

merged

mentioned in commit db4bbbab

We decided to move !11170 (merged) to %9.3, so I guess this should be, too.

changed milestone to %9.3