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.

Sign up for GitHub

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

Jump to bottom

fix concurrent TimeClock issue #756

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

fix concurrent TimeClock issue #756

wants to merge 2 commits into from

Conversation

bseber
Copy link
Contributor

@bseber bseber commented Jul 3, 2024 •
edited
Loading

fixes #765 (related #755)

Here are some things you should have thought about:

Multi-Tenancy

  • Extended new entities with AbstractTenantAwareEntity?
  • New entity added to TenantAwareDatabaseConfiguration?
  • Tested with dev-multitenant profile?

refs #755

@bseber bseber force-pushed the 755-fix-time-clock-start-doubleclick-issue branch 2 times, most recently from a94c473 to 7b0dd29 Compare July 3, 2024 20:40
@bseber bseber mentioned this pull request Jul 5, 2024
Merged
derTobsch added a commit that referenced this pull request Jul 5, 2024
@derTobsch
prevent double click form submit (#757)
5553ff3 
fixes #755

quickfix...  
see #756 for the
root issue
@bseber bseber force-pushed the 755-fix-time-clock-start-doubleclick-issue branch from 7b0dd29 to 9fee0f9 Compare July 5, 2024 09:34
@bseber bseber added this to the 2.x milestone Jul 5, 2024
@github-actions github-actions bot added the status: planned This issue is planned in a milestone and won't be touched by stale bot. label Jul 5, 2024
@bseber bseber force-pushed the 755-fix-time-clock-start-doubleclick-issue branch 2 times, most recently from 741b943 to 46f7838 Compare July 5, 2024 09:43
@derTobsch derTobsch force-pushed the 755-fix-time-clock-start-doubleclick-issue branch from 46f7838 to c629554 Compare July 19, 2024 07:59
@bseber bseber force-pushed the 755-fix-time-clock-start-doubleclick-issue branch 2 times, most recently from f15a42d to 6fe7ebb Compare July 26, 2024 16:26
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jul 26, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@github-actions github-actions bot added the Stale label Sep 25, 2024
@github-actions github-actions bot closed this Oct 3, 2024
@derTobsch derTobsch reopened this Oct 3, 2024
@github-actions github-actions bot removed the Stale label Oct 4, 2024
@derTobsch derTobsch force-pushed the 755-fix-time-clock-start-doubleclick-issue branch from 6fe7ebb to e81bf18 Compare October 4, 2024 08:31
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Oct 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
status: planned This issue is planned in a milestone and won't be touched by stale bot.
Projects
None yet
Milestone
2.x
Development

Successfully merging this pull request may close these issues.

Stoppuhr starten/beenden ist nicht Transaktionssicher
2 participants
@bseber @derTobsch