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

Added hidden flag functionality to activities #281

Merged
merged 9 commits into from
Jan 15, 2025
Merged

Conversation

Muncherkin
Copy link
Contributor

Made it possible to hide activities using a hidden flag. Also added functionality to query all activities including or excluding hidden ones.

Copy link

github-actions bot commented Dec 23, 2024

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 67.94% 1585/2333
🔴 Branches 47.28% 261/552
🔴 Functions 57.26% 276/482
🟡 Lines 67.6% 1504/2225

Test suite run success

256 tests passing in 22 suites.

Report generated by 🧪jest coverage report action from e4a79d6

@Muncherkin
Copy link
Contributor Author

JAG ÄR JESUS! DET ÄR LÖST! MERGEA MIG!

src/api/activity.api.ts Outdated Show resolved Hide resolved
src/schemas/activity.graphql Show resolved Hide resolved
src/schemas/activity.graphql Show resolved Hide resolved
@Muncherkin Muncherkin requested a review from afroborg January 4, 2025 17:22
@Muncherkin
Copy link
Contributor Author

Reran the workflow but the Test annotations persisted even though the tests did actually work after the rerun

Copy link
Member

@afroborg afroborg left a comment

Choose a reason for hiding this comment

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

nice, just one small suggestion

src/schemas/activity.graphql Outdated Show resolved Hide resolved
@Muncherkin Muncherkin merged commit a8c38ce into main Jan 15, 2025
7 checks passed
Copy link

I have removed this deploy now 😇

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