Skip to content

add internal descriptions to worktimes (63818) #229

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

Merged
merged 2 commits into from
Apr 24, 2025

Conversation

svenwey
Copy link
Collaborator

@svenwey svenwey commented Feb 19, 2025

these internal descriptions are displayed in puzzletime, and in the csv export, but not in the time rapport

@svenwey svenwey requested a review from Kagemaru February 19, 2025 10:53
@svenwey svenwey self-assigned this Feb 19, 2025
@svenwey svenwey requested review from daniel-illi and removed request for Kagemaru February 26, 2025 10:31
@daniel-illi daniel-illi changed the title add internal desciptions to worktimes add internal desciptions to worktimes (63818) Feb 26, 2025
@svenwey svenwey requested a review from kronn March 5, 2025 07:17
@kronn kronn changed the title add internal desciptions to worktimes (63818) add internal descriptions to worktimes (63818) Mar 6, 2025
@svenwey
Copy link
Collaborator Author

svenwey commented Mar 10, 2025

@kronn when migrating the db, the schema is reformatted in quite a few places. Are these changes okay or was there some misconfiguration on my side?

@kronn
Copy link
Member

kronn commented Mar 13, 2025

@kronn when migrating the db, the schema is reformatted in quite a few places. Are these changes okay or was there some misconfiguration on my side?

Can be okay. Are you using the same DB-version as in production, on CI and as everyone else? If so, then yes, it is okay. If there are differences in the setup, these should be addressed first.

@svenwey
Copy link
Collaborator Author

svenwey commented Mar 17, 2025

As far as I know, it should be the same

@svenwey
Copy link
Collaborator Author

svenwey commented Mar 21, 2025

However, I would be thankful if this could be verified and some other person can generate the schema themselves. Just to make sure that my setup was correct.

@Kagemaru Kagemaru force-pushed the feature/63818-add_field_internal_remarks_ branch 2 times, most recently from 1f67f58 to c200487 Compare April 24, 2025 11:44
these internal descriptions are displayed in puzzletime, and in the csv export, but not in the time rapport
@Kagemaru Kagemaru force-pushed the feature/63818-add_field_internal_remarks_ branch from c200487 to 94b93df Compare April 24, 2025 11:51
@Kagemaru Kagemaru merged commit 24ab55b into master Apr 24, 2025
3 checks passed
@Kagemaru Kagemaru deleted the feature/63818-add_field_internal_remarks_ branch April 24, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants