chore: Clean hash urls

[shubham-fyle]

Clickup
https://app.clickup.com/

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

Walkthrough

Test expectations updated to reflect a change in admin expense view URL format. Two test cases were modified to expect URLs without the leading hash fragment (from .../#/view_expense/... to .../view_expense/...). Lines changed: 2 added, 2 removed. No functional code alterations.

Changes

Cohort / File(s)	Summary
Test Expectations Update tests/test_xero/test_tasks.py	Updated two test assertions to expect admin expense view URLs without the leading hash fragment, changing URL format from /#/view_expense/ to /view_expense/.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hash fragment vanishes in the night,
Tests align with URLs made right,
Two little tweaks, so clean and bright,
The rabbit hops with pure delight! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description only contains a ClickUp link with no meaningful explanation of the changes, making it too vague to convey information about the changeset.	Add a brief description explaining what the PR accomplishes, such as 'Removes hash fragments from admin expense view URLs in test expectations'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'chore: Clean hash urls' directly relates to the changeset, which removes hash fragments from URLs in test expectations.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch remove-hash-urls-to-web-app-final

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
apps
exceptions.py	54	9	83%	30, 42, 67, 73, 91, 104, 122–124
apps/fyle
actions.py	141	7	95%	92, 314, 353–357
enums.py	27	0	100%
helpers.py	133	8	94%	61, 225, 232–235, 238–239
models.py	270	10	96%	375–377, 381–383, 402, 406, 462, 589, 611, 717
queue.py	48	0	100%
serializers.py	21	0	100%
signals.py	24	0	100%
tasks.py	439	23	95%	146–147, 154–156, 346, 356–365, 373, 377, 701–703, 721–738, 750–752
views.py	79	0	100%
apps/internal
tasks.py	53	7	87%	41–46, 73
apps/mappings
actions.py	36	4	89%	46–47, 50–51
constants.py	2	0	100%
exceptions.py	104	12	88%	38–40, 43–44, 47–48, 55–57, 60–61
helpers.py	21	7	67%	17–21, 28–30
models.py	31	0	100%
queue.py	35	16	54%	57–123
schedules.py	12	0	100%
serializers.py	10	0	100%
signals.py	80	9	89%	39–40, 117–119, 144–150
tasks.py	35	1	97%	83
utils.py	34	20	41%	46–113
views.py	32	0	100%
apps/tasks
enums.py	15	0	100%
models.py	63	0	100%
serializers.py	6	0	100%
views.py	9	0	100%
apps/users
helpers.py	12	0	100%
models.py	53	0	100%
apps/workspaces
actions.py	150	41	73%	141–142, 163–172, 201–273
email.py	36	0	100%
helpers.py	45	1	98%	25
models.py	136	1	99%	127
permissions.py	20	1	95%	23
queue.py	41	6	85%	60–64, 87
serializers.py	26	0	100%
signals.py	28	0	100%
tasks.py	124	2	98%	157–158
utils.py	100	37	63%	94–231, 263–264
views.py	109	0	100%
apps/workspaces/apis/advanced_settings
serializers.py	69	5	93%	163, 166, 172, 178, 181
triggers.py	12	0	100%
views.py	11	0	100%
apps/workspaces/apis/clone_settings
helpers.py	9	0	100%
serializers.py	44	0	100%
views.py	22	2	91%	14–16
apps/workspaces/apis/errors
serializers.py	20	0	100%
views.py	19	0	100%
apps/workspaces/apis/export_settings
helpers.py	72	3	96%	132–134
serializers.py	94	6	94%	211, 214, 217, 224, 231, 240
triggers.py	19	0	100%
views.py	11	0	100%
apps/workspaces/apis/import_settings
serializers.py	79	3	96%	226, 229, 232
triggers.py	63	3	95%	36–37, 89
views.py	11	0	100%
apps/workspaces/templatetags
custom_filters.py	5	0	100%
apps/xero
actions.py	30	8	73%	22–32
exceptions.py	141	25	82%	54–55, 98–135, 163–169, 181, 189–195
models.py	216	5	98%	40–44, 55, 376, 395
queue.py	112	1	99%	37
serializers.py	15	0	100%
tasks.py	493	91	82%	145–169, 182, 274–278, 289–290, 301, 331, 354–355, 463–467, 478–479, 534, 561–562, 701, 758–760, 764–765, 906–911, 945–955, 958–969, 973–1004, 1037–1041, 1068–1072, 1091, 1098–1102
utils.py	338	26	92%	100, 139–184, 215, 319, 328–349, 364, 752, 831, 882
views.py	96	0	100%
workers
actions.py	26	0	100%
helpers.py	46	0	100%
worker.py	56	3	95%	76–77, 124
TOTAL	4823	403	92%

Tests	Skipped	Failures	Errors	Time
308	0 💤	0 ❌	0 🔥	1m 33s ⏱️

[github-actions]

---

Diff Coverage
Diff: origin/master..HEAD, staged and unstaged changes

No lines with coverage information in this diff.