Skip to content

toggle hs deltagere lagt til#2988

Open
miove4372 wants to merge 2 commits into
mainfrom
mikaeloverland/add-admin-dashboard
Open

toggle hs deltagere lagt til#2988
miove4372 wants to merge 2 commits into
mainfrom
mikaeloverland/add-admin-dashboard

Conversation

@miove4372
Copy link
Copy Markdown
Collaborator

bilde

@miove4372 miove4372 force-pushed the mikaeloverland/add-admin-dashboard branch 3 times, most recently from 8292a9b to 80d0c18 Compare April 8, 2026 15:26
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 8, 2026

Preview Environment

Project URL
Frontend https://pr-2988.web.echo-webkom.no
Backend https://pr-2988.uno.echo-webkom.no
Status 🟢 Deployed

Deployed from bd31ee8 at 29.4.2026, 17:22:55

@miove4372 miove4372 force-pushed the mikaeloverland/add-admin-dashboard branch from 80d0c18 to 351a0c8 Compare April 8, 2026 15:55
Comment thread apps/web/src/app/(default)/admin/toggle-components/toggle-hs-applications.ts Outdated
@omfj
Copy link
Copy Markdown
Member

omfj commented Apr 8, 2026

Bra jobba. Litt so ikke trenger å være med som vi kan fjerne.

@omfj omfj force-pushed the mikaeloverland/add-admin-dashboard branch from 351a0c8 to 291369f Compare April 8, 2026 17:19
@johanneherheim johanneherheim force-pushed the mikaeloverland/add-admin-dashboard branch from 291369f to 18da36f Compare April 29, 2026 15:04
@omfj
Copy link
Copy Markdown
Member

omfj commented Apr 29, 2026

Har du pushet feil @johanneherheim eller er du med på den her nå?

import { featureFlags } from "../../../../data/kv/namespaces";

export default async function toggleHsApplications(newValue: boolean) {
console.info("Setting HS-Applications to:", newValue);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Gjerne fjern console.log.

await featureFlags.set("HS-Application", { showHSApplications: newValue });

const check = await featureFlags.get("HS-Application");
console.info("Changed to", check);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
console.info("Changed to", check);

Fjern her og

Comment on lines +17 to +19
schema: z.object({
showHSApplications: z.boolean(),
}),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
schema: z.object({
showHSApplications: z.boolean(),
}),
schema: z.boolean().default(false),

Tror ikke vi trenger å gjøre det til et objekt.

@johanneherheim
Copy link
Copy Markdown
Member

Har du pushet feil @johanneherheim eller er du med på den her nå?

skulle berre rebase?

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.

3 participants