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

Add a new SettingUpdated event for Epicea Monitor. #17350

Merged
merged 4 commits into from
Nov 12, 2024

Conversation

hernanmd
Copy link
Member

@hernanmd hernanmd commented Nov 7, 2024

This PR introduces a simple SettingUpdated event object to support logging for the new settings browser.

Notes

  • It uses the Epicea package to record events and get notifications.
  • Setting update events are not serialized using STON since each setting is recorded, and some instance variables store blocks which are not yet serializable by STON writer.

src/Epicea/EpCangedSettingEvent.class.st Outdated Show resolved Hide resolved
src/Epicea/EpMonitor.class.st Outdated Show resolved Hide resolved
@hernanmd hernanmd requested a review from jecisc November 8, 2024 15:41
@Ducasse
Copy link
Member

Ducasse commented Nov 9, 2024

Hi hernan
I was wondering what is the scenario that you have in mind?
Why log setting changes?

@hernanmd
Copy link
Member Author

Hi hernan I was wondering what is the scenario that you have in mind? Why log setting changes?

Sure, here are some use cases:

  • When users encounter issues, logs provide a historical record of changes, so we can identify what settings were modified leading up to the error.
  • We can replicate the environment and conditions under which a problem occurred.
  • Understanding user behavior and preferences over time.
  • It can act as a form of version control for configurations, allowing users to revert to previous settings if needed.
  • We can analyze how specific settings affect performance metrics over time. This is particularly useful in optimizing IDE performance
  • For UX we can offer personalized experiences based on past behavior

@jecisc jecisc merged commit 38f7d27 into pharo-project:Pharo13 Nov 12, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants