lorentz/wulf-pulse
1
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

fix: correct Autotask URL format and strip bracket artifacts from link rendering

Browse source
This commit is contained in:
lorentz 2026-03-23 11:24:54 -04:00
parent 21a41ef8ef
commit be61c30204
1 changed files with 11 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

15
components/quotes/ticket-detail-modal.tsx
View file

@ -34,7 +34,7 @@ import {
// noteType values that are system/automated (workflow rules, monitoring, auto-close, etc.)
const SYSTEM_NOTE_TYPES = new Set([13, 91, 93, 94, 99, 101]);
const URL_REGEX = /https?:\/\/[^\s<>"')]+/g;
const URL_REGEX = /https?:\/\/[^\s<>"'\[\]()]+/g;
function renderTextWithLinks(text: string): React.ReactNode[] {
const parts: React.ReactNode[] = [];
@ -43,9 +43,11 @@ function renderTextWithLinks(text: string): React.ReactNode[] {
URL_REGEX.lastIndex = 0;
while ((match = URL_REGEX.exec(text)) !== null) {
if (match.index > lastIndex) {
parts.push(text.slice(lastIndex, match.index));
// Strip orphaned bracket that wraps the URL e.g. "link [https://..."
const before = text.slice(lastIndex, match.index).replace(/\s*\[$/, ' ');
if (before) parts.push(before);
}
const url = match[0].replace(/[.,;:!?)]+$/, '');
const url = match[0].replace(/[.,;:!?]+$/, '');
let label: string;
try {
label = new URL(url).hostname.replace(/^www\./, '');
@ -64,6 +66,8 @@ function renderTextWithLinks(text: string): React.ReactNode[] {
</a>
);
lastIndex = match.index + match[0].length;
// Consume trailing bracket/paren that closed around the URL
if (lastIndex < text.length && /^[\])]/.test(text[lastIndex])) lastIndex++;
}
if (lastIndex < text.length) parts.push(text.slice(lastIndex));
return parts;
@ -450,7 +454,10 @@ export function TicketDetailModal({ ticketNumber, open, onOpenChange }: TicketDe
<Button
size="sm"
onClick={() => {
window.open(`https://wam.autotask.net/Mvc/ServiceDesk/TicketDetail.mvc?ticketId=${ticket.id}`, '_blank');
window.open(
`https://ww1.autotask.net/Mvc/ServiceDesk/TicketDetail.mvc?workspace=False&ids%5B0%5D=${ticket.id}&ticketId=${ticket.id}`,
'_blank'
);
}}
>
<ExternalLink className="h-4 w-4 mr-2" />