πŸ‡ΊπŸ‡ΈUnited States @jvest

Account created on 20 October 2011, over 12 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States jvest

Adding clarification on testing/review:

Used a shared DrupalPod instance
Enabled Authored By
Added several users added so that the list will be long enough when expanded

The overlay issue appearing behind the table heading is fixed.
Also manually changed the z-index back to original to show the original problem of the drop down appearing behind the table heading.

πŸ‡ΊπŸ‡ΈUnited States jvest

Have reviewed the merge request 2.0.x MR !4 and tested the diff as a patch, steps were:

Reproduced Issue:
Used existing Drupal 10.2 install
Installed usagov module
Configured with user agent and auth key in services/usajobs
Set to Organization = 11th Wing (known to not have any jobs to display)
Added block to homepage sidebar.
Resulted in the double periods being displayed:

Test For merge diff to fix:
Added the merge diff to composer.json:
"drupal/usajobs": {
"3439485: if (empty($markup)) sentence is grammatically incorrect. (Double Period)":
"https://git.drupalcode.org/project/usajobs/-/merge_requests/4.diff"
},

Run composer install.
Results in only 1 period being displayed.

Test is successful.

πŸ‡ΊπŸ‡ΈUnited States jvest

Reviewed #27 and tested with @kd_ace, issue looks good for rtbc status.

Production build 0.69.0 2024