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 table for sizzle time tracking project #2617

Merged
merged 8 commits into from
Aug 9, 2024

Conversation

JisanAR03
Copy link
Contributor

This pull request introduces new models and updates to support enhanced time tracking capabilities within our application, specifically focusing on tracking user activities and screenshots associated with GitHub issues. These changes aim to improve project management by providing detailed insights into how much time is spent on specific issues and what activities are undertaken during that time.

@DonnieBLT sir , can you please review this PR?

@JisanAR03 JisanAR03 requested a review from DonnieBLT August 8, 2024 14:15
@DonnieBLT
Copy link
Collaborator

Looks good! I think we have an issues table we can use or build on and I think it will be good to generalize the table names to remove sizzle. Also use created as the date name and make sure they all have it.

Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

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

Looks good, some tweaks requested. Thank you!

website/models.py Outdated Show resolved Hide resolved
@DonnieBLT
Copy link
Collaborator

Sorry, we should have a GithubIssue model - they are different than Issue. Also lets skip screenshots for now can you also please flatten the migration or have just one migration

@JisanAR03
Copy link
Contributor Author

Sorry, we should have a GithubIssue model - they are different than Issue. Also lets skip screenshots for now can you also please flatten the migration or have just one migration

ok sure , I will update

@DonnieBLT DonnieBLT enabled auto-merge (squash) August 9, 2024 20:04
@DonnieBLT DonnieBLT merged commit 02d940f into OWASP-BLT:main Aug 9, 2024
8 checks passed
@JisanAR03 JisanAR03 mentioned this pull request Aug 9, 2024
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