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

Feature/sac 287 add family member #48

Merged
merged 4 commits into from
Mar 22, 2024

Conversation

daniel-illi
Copy link
Member

@daniel-illi daniel-illi force-pushed the feature/sac-287-add-family-member branch from c6dee59 to e189595 Compare February 28, 2024 13:01
@daniel-illi daniel-illi changed the title Fix PeopleManager manager_and_managed_the_same validation message Feature/sac 287 add family member Feb 28, 2024
@daniel-illi daniel-illi force-pushed the feature/sac-287-add-family-member branch from 4f67381 to 444a436 Compare February 29, 2024 07:38
@amaierhofer amaierhofer force-pushed the feature/sac-287-add-family-member branch from 444a436 to 002ab21 Compare March 1, 2024 13:40
@daniel-illi daniel-illi force-pushed the feature/sac-287-add-family-member branch 3 times, most recently from 251e8de to 4c0a19d Compare March 7, 2024 15:39
@daniel-illi daniel-illi force-pushed the feature/sac-287-add-family-member branch from 4c0a19d to 55ed101 Compare March 15, 2024 10:16
redirect_to redirect_to_path
else
render :new
ActiveRecord::Base.transaction do
Copy link
Member

Choose a reason for hiding this comment

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

Wenn die Transaktion fehlschlägt, was wird dann gerendert? Fehlt hier nicht noch ein rescue oder so?

@daniel-illi daniel-illi changed the base branch from master to sac-master March 21, 2024 15:01
@daniel-illi daniel-illi force-pushed the feature/sac-287-add-family-member branch from 55ed101 to 5adbfe1 Compare March 21, 2024 15:50
@daniel-illi daniel-illi merged commit eb0c4e3 into sac-master Mar 22, 2024
3 of 5 checks passed
@daniel-illi daniel-illi deleted the feature/sac-287-add-family-member branch March 22, 2024 10:32
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.

PEOPLE: Person in Familie hinzufügen
2 participants