Comments url should support alphanumeric slug

[actlikewill]

some models eg country in countries_plus have letters in the pk
https://lawsafrica.sentry.io/issues/5914901445/events/0fb1e238b89f43ecb51e8b6b1bfb2d72/

[github-actions]

Test Results

2 tests  +1   2 ✅ +1   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit d719e32. ± Comparison against base commit ee7177f.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

africanlii.tests.test_views.AfricanliiViewsTest ‑ test_homepage

liiweb.tests.test_mnc_matcher.MncMatcherTest ‑ test_html_matches
liiweb.tests.test_mnc_matcher.MncMatcherTest ‑ test_za_provincial_matches

[github-actions]

Test Results

50 tests  +49   50 ✅ +49   10s ⏱️ +10s
13 suites +12    0 💤 ± 0 
13 files   +12    0 ❌ ± 0 

Results for commit d719e32. ± Comparison against base commit ee7177f.

This pull request removes 1 and adds 50 tests. Note that renamed tests count towards both.

africanlii.tests.test_views.AfricanliiViewsTest ‑ test_homepage

peachjam.tests.test_admin.TestJudgmentAdmin ‑ test_add_judgment_docx_swap_pdf
peachjam.tests.test_admin.TestJudgmentAdmin ‑ test_add_judgment_pdf_swap_docx
peachjam.tests.test_bulk_import.JudgmentBulkImportTestCase ‑ test_case_number_import_without_matter_type
peachjam.tests.test_bulk_import.JudgmentBulkImportTestCase ‑ test_case_numbers_with_matter_type
peachjam.tests.test_bulk_import.JudgmentBulkImportTestCase ‑ test_import_with_taxonomy
peachjam.tests.test_bulk_import.JudgmentBulkImportTestCase ‑ test_judgment_bulk_import
peachjam.tests.test_bulk_import.JudgmentBulkImportTestCase ‑ test_source_file_prefers_docx_over_pdf
peachjam.tests.test_citations.CitationAnalyserTestCase ‑ test_delete_citations
peachjam.tests.test_citations.CitationAnalyserTestCase ‑ test_delete_citations_should_not_change_akn
peachjam.tests.test_citations.CitationAnalyserTestCase ‑ test_pdf_extractions
…

♻️ This comment has been updated with latest results.