Skip to content

[AJDA-1176] Skip orphaned manifests#129

Merged
ondrajodas merged 4 commits intomasterfrom
AJDA-1176-ondra
Oct 6, 2025
Merged

[AJDA-1176] Skip orphaned manifests#129
ondrajodas merged 4 commits intomasterfrom
AJDA-1176-ondra

Conversation

@ondrajodas
Copy link
Copy Markdown
Contributor

@ondrajodas ondrajodas commented Oct 3, 2025

https://keboola.atlassian.net/browse/AJDA-1176

buď to fixneme takto - až ve chvíli kdy jsou nějaká data tak vytvoříme manifest a nebo druhá možnost že ten datový soubor vytvoříme hned tady https://github.com/keboola/google-analytics-extractor/blob/master/src/Extractor/Output.php#L164 a taky by se to tím vyřešilo

return $config;
}

public function testAppRunDailyWalk(): void
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

GA4 už nejsou... takže jejich testy nebudou fungovat - pryč s tím

Copy link
Copy Markdown

@sykora-ji sykora-ji left a comment

Choose a reason for hiding this comment

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

Za mě to vypadá dobře

Copy link
Copy Markdown

@ErikZigo ErikZigo left a comment

Choose a reason for hiding this comment

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

🤔

}

public function paginate(array $query, array $report, CsvFile $csvFile): void
public function paginate(array $query, array $report, CsvFile $csvFile): int
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

A testicek nic?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no jo, máš ho tam... dal jsem ho jen k tomu PropertiesPaginator, protože ten Profiles se jednoho krásnýho dne smaže - google už nepodporuje tyhle starý reporty a už se na ně nedoboucháš

@ondrajodas ondrajodas requested a review from ErikZigo October 3, 2025 13:01
Copy link
Copy Markdown

@ErikZigo ErikZigo left a comment

Choose a reason for hiding this comment

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

lgtm

@ondrajodas ondrajodas merged commit 308435b into master Oct 6, 2025
1 check passed
@ondrajodas ondrajodas deleted the AJDA-1176-ondra branch October 6, 2025 09:09
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