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

[fix] Include all modified files in backup #206 #207

Merged
merged 2 commits into from
May 25, 2024
Merged

Conversation

okraits
Copy link
Member

@okraits okraits commented Apr 11, 2024

Fixes #206

@okraits okraits requested a review from nemesifier April 12, 2024 06:06
@okraits
Copy link
Member Author

okraits commented May 2, 2024

@nemesifier Anything keeping this bugfix for the severe issue from getting merged?

@nemesifier nemesifier added the bug label May 20, 2024
@nemesifier nemesifier added this to In progress in OpenWISP Priorities for next releases via automation May 20, 2024
@nemesifier
Copy link
Member

@okraits sorry it slipped from my attention, I have added it to my todo list.

The only small issue I noticed from the issue description is that it's not 100% clear to me how to replicate the issue for testing purposes, if you could provide a series of steps and sample configurations, it would make the review process faster.

@okraits
Copy link
Member Author

okraits commented May 23, 2024

@nemesifier I updated the issue description to include a detailed example of how to reproduce the issue: #206 (comment)

Copy link
Member

@nemesifier nemesifier left a comment

Choose a reason for hiding this comment

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

I was able to replicate the bug and I confirm the fix works.
Thanks @okraits! 👍👍👍

OpenWISP Priorities for next releases automation moved this from In progress to Reviewer approved May 25, 2024
@nemesifier nemesifier merged commit 05cfe4e into master May 25, 2024
2 checks passed
OpenWISP Priorities for next releases automation moved this from Reviewer approved to Done May 25, 2024
@nemesifier nemesifier deleted the fix-backup branch May 25, 2024 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

[bug] Not all modified files are included in the backup used to restore the last working configuration
2 participants