fix: open alert url encoding

[00041275]

close #4422

[vercel]

@00041275 is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
keep	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 12, 2025 10:26am

[shahargl]

lgtm

[shahargl]

@00041275 there is a unit test that keep failing:

FAILED tests/test_alert_dto.py::test_alert_dto_url_encoding - AssertionError: assert AnyHttpUrl('h...al%5D-Down', ) == '[https://plat...ernal%5D-Down](https://plat...ernal]-down/)'

[shahargl]

see test failing

FAILED tests/test_alert_dto.py::test_alert_dto_url_encoding - AssertionError: assert AnyHttpUrl('h...al%5D-Down', ) == '[https://plat...ernal%5D-Down](https://plat...ernal]-down/)'

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 46.64%. Comparing base (a33c34f) to head (e481fdf).
Report is 52 commits behind head on main.

Files with missing lines	Patch %	Lines
keep/api/models/alert.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #4501       +/-   ##
===========================================
+ Coverage   31.11%   46.64%   +15.52%     
===========================================
  Files          93      164       +71     
  Lines       10445    16728     +6283     
===========================================
+ Hits         3250     7802     +4552     
- Misses       7195     8926     +1731     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[talboren]

@00041275 is this still relevant? can you fix the failing test?

[00041275]

@00041275 is this still relevant? can you fix the failing test?

yes, i will check it
please wait

[shahargl]

@00041275 hey! Any update? we usually do not like keeping stale PR's. if you want we can close and you will reopen when its ready?

[talboren]

Tests are still failing. Closing this for now until we fix it. cc @tuantran0910