Skip to content

Remove exact date from varslet frist#6

Open
cskrov wants to merge 1 commit into
mainfrom
remove-exact-date
Open

Remove exact date from varslet frist#6
cskrov wants to merge 1 commit into
mainfrom
remove-exact-date

Conversation

@cskrov

@cskrov cskrov commented Mar 14, 2025

Copy link
Copy Markdown
Contributor

Diskuter med MIa først.

Copilot AI review requested due to automatic review settings March 14, 2025 15:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request removes the exact date from the varslet output as part of updating the displayed information in the SakPage. Key changes include the removal of the temporary “varslet” variable, inlining the date format call for the null unit type case, and updating the final return string to no longer include the exact date.

Comments suppressed due to low confidence (1)

app/[lang]/saker/[id]/page.tsx:142

  • Ensure that removing the exact date (previously included as 'varslet') from the final output aligns with the intended business logic. If this change is desired, consider updating any related tests and documentation.
return `${unit} ${FROM[lang]} ${mottatt}`;

@cskrov cskrov force-pushed the remove-exact-date branch from dd6a8b0 to f47d29d Compare March 20, 2025 14:04
@cskrov cskrov force-pushed the remove-exact-date branch from f47d29d to 363e1f2 Compare August 18, 2025 13:35
@cskrov cskrov force-pushed the remove-exact-date branch from 363e1f2 to b2ef5b3 Compare August 29, 2025 14:46
@cskrov cskrov force-pushed the remove-exact-date branch from b2ef5b3 to 23de82f Compare December 2, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants