Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSCEXAM-1396 Add titles to teacher and admin tab title #1190

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Conversation

VirmasaloA
Copy link
Collaborator

No description provided.

@@ -31,6 +31,7 @@ export const APP_ROUTES: Route[] = [
{
path: 'logout',
component: LogoutComponent,
title: () => buildTitle('i18n_logout_title'),
Copy link
Collaborator Author

@VirmasaloA VirmasaloA Jan 30, 2025

Choose a reason for hiding this comment

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

Tää on periaatteessa turha, koska tää sivu näkyy ruudulla noin 20 millisekuntia

@@ -115,7 +116,7 @@ export const APP_ROUTES: Route[] = [
path: 'enrolments/:id',
loadComponent: () =>
import('./enrolment/exams/exam-enrolments.component').then((mod) => mod.ExamEnrolmentsComponent),
title: (route) => buildTitle('i18n_enrolment_title', `#${route.params.id}`),
title: () => buildTitle('i18n_enrolment_title'),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Totesin numeroinnin ylimääräiseksi niin poistin sen kaikista kohista kokonaan

const extra = extraPart ? ` ${extraPart}` : '';
return tx.get(key).pipe(map(() => `${tx.instant(key)}${extra} - EXAM`));
};

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oisko tän voinu tehä niin että funktion ois hakenu app.routes filusta?

Copy link
Collaborator

@lupari lupari left a comment

Choose a reason for hiding this comment

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

Käännöksissä on jotain outoja välilyöntimuutoksia. Muuten näyttää ok:lta.

@VirmasaloA
Copy link
Collaborator Author

Kielitiedostoissa oli 1. ylimääräsiä välilyöntejä 2. erikoismerkkinä merkattuja välilyöntejä. AI halus välttämättä korjaa ne ja oon sen kanssa samaa mieltä. Oon joskus korjannu niitä manuaalisesti, mut nyt ku se olli automaattista nii se oli kivan helppo suoritus.

@VirmasaloA VirmasaloA merged commit 806769a into dev Jan 31, 2025
5 checks passed
@VirmasaloA VirmasaloA deleted the CSCEXAM-1396 branch January 31, 2025 07:35
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