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

feat: delete obsolete answers after 5 years (hitobito_youth#58) #120

Merged
merged 4 commits into from
Sep 30, 2024

Conversation

diegosteiner
Copy link
Contributor

@diegosteiner diegosteiner commented Sep 23, 2024

Copy link
Member

@kronn kronn left a comment

Choose a reason for hiding this comment

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

Code scheint okay. Soll das wirklich noch in den Release mit rein?

// @ThomasEllenberger

@richardjubla
Copy link
Contributor

Das war unsere Vorstellung: hitobito/hitobito_youth#58 (comment)

@diegosteiner
Copy link
Contributor Author

Das war unsere Vorstellung: hitobito/hitobito_youth#58 (comment)

@richardjubla ich weiss nicht wie ich auf die 5 Monate gekommen bin, aber jetzt sollte es stimmen, oder?

@Largo Largo changed the title feat: delete obsolete answers after 5 months (hitobito_youth#58) feat: delete obsolete answers after 5 years (hitobito_youth#58) Sep 27, 2024
@Largo
Copy link
Contributor

Largo commented Sep 27, 2024

@diegosteiner
specs schlagen gerade noch fehl.

undefined method `customize_derived' for #<Event::Question::AhvNumber id: 1, event_id: nil, multiple_choices: false, admin: false, disclosure: nil, type: "Event::Question::AhvNumber", derived_from_question_id: nil, event_type: nil, question: "AHV-Nummer?", choices: nil>

@diegosteiner diegosteiner merged commit a528746 into master Sep 30, 2024
9 checks passed
@diegosteiner diegosteiner deleted the feature/hitobito_youth#58 branch September 30, 2024 08:02
@richardjubla
Copy link
Contributor

Getestet auf dem produktiven System:

image
image
Hier scheinen die Anpassungen nicht zu "ziehen"

Auf der Stage-Umgebung habe ich etwas gesucht und keine Bemerkungen oder Event-Fragen mit Daten vor 60 Monaten gefunden.

@ThomasEllenberger
Copy link

Getestet auf dem produktiven System:

image image Hier scheinen die Anpassungen nicht zu "ziehen"

Auf der Stage-Umgebung habe ich etwas gesucht und keine Bemerkungen oder Event-Fragen mit Daten vor 60 Monaten gefunden.

Habe kurz genau genau dieses Event überprüft. Dies scheint nun alles gelöscht zu sein.
Kann es sein, dass dies etwas brauchte bis dies gelöscht war, oder die Changes noch nicht aktiv waren zum Zeitpunkt des Tests?
Kannst du evtl. erneut ein paar Stichproben machen, ob die automatische Löschung nun korrekt funktioniert, oder du noch immer daten findest die nicht gelöscht worden sind.

@richardjubla
Copy link
Contributor

Habe kurz genau genau dieses Event überprüft. Dies scheint nun alles gelöscht zu sein. Kann es sein, dass dies etwas brauchte bis dies gelöscht war, oder die Changes noch nicht aktiv waren zum Zeitpunkt des Tests? Kannst du evtl. erneut ein paar Stichproben machen, ob die automatische Löschung nun korrekt funktioniert, oder du noch immer daten findest die nicht gelöscht worden sind.

Auf dem Produktivsystem als funktionierend getestet.

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.

5 participants