Skip to content

Update site deletion test#6101

Open
aerosol wants to merge 1 commit intomasterfrom
update-site-deletion-test
Open

Update site deletion test#6101
aerosol wants to merge 1 commit intomasterfrom
update-site-deletion-test

Conversation

@aerosol
Copy link
Member

@aerosol aerosol commented Feb 26, 2026

No description provided.

@aerosol aerosol force-pushed the update-site-deletion-test branch from 0bec9e0 to 8e22e24 Compare February 26, 2026 09:32

on_ee do
{:ok, g1} = Plausible.Goals.create(site, %{"page_path" => "/1"})
{:ok, g2} = Plausible.Goals.create(site, %{"event_name" => "/2"})
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{:ok, g2} = Plausible.Goals.create(site, %{"event_name" => "/2"})
{:ok, g2} = Plausible.Goals.create(site, %{"event_name" => "Signup"})

a bit more realistic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants