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

Issue 254/big brother report #261

Merged
merged 5 commits into from
Nov 5, 2024
Merged

Issue 254/big brother report #261

merged 5 commits into from
Nov 5, 2024

Conversation

rphovley
Copy link
Member

@rphovley rphovley commented Nov 1, 2024

The Pull Request is ready

  • all github actions are passing
  • are changes backwards compatible?
  • fixes Add Big Brother Weekly Report Option #254
  • the branch follows the naming schema issue-123-enable-x-does-not-disable-y
  • the pull request has a sensible title

Intention

With this change I intend to add the big brother report to the reports page

Review Points

The code follows best practices

  • the code is readable
  • issues for follow-up tasks have been created
  • there is no any type used

Notes

Copy link
Contributor

github-actions bot commented Nov 1, 2024

Visit the preview URL for this PR (updated for commit c0b587b):

https://codeclimbersio--pr261-issue-254-big-brothe-79hc2g8n.web.app

(expires Fri, 08 Nov 2024 20:26:25 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 3c28ab1d661a7526dc2530b2448a1aa4f88d23d4

@rphovley rphovley merged commit e73b25a into main Nov 5, 2024
10 checks passed
@rphovley rphovley deleted the issue-254/big-brother-report branch November 5, 2024 00:59
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.

Add Big Brother Weekly Report Option
1 participant