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

Manufacturer name in filename in autosave #1080

Merged
merged 7 commits into from
Jul 24, 2024

Conversation

mateusz-szafraniec
Copy link

What's new

  • Manufacturer name in filename in autosave
  • removed underscores in date and time in filename

Verification

  • enable autosave in SubGhz
  • all signals received will be saved with manufacturer name i.e. KL_BENINCA_xxxx-date-time

Checklist (For Reviewer)

  • PR has description of feature/bug
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix
  • [ ]

@RogueMaster
Copy link
Owner

You are removing the protocol name from the name for the sake of having the manufacturer name instead? I think the protocol used may be better to have, let me know why I should think otherwise...

@mateusz-szafraniec
Copy link
Author

Of course:
This helps keep files in order, facilitates their cataloging and subsequent analysis.
Especially if you use the device in an environment where many systems work and you are looking for the right one.
The protocol name has not been removed, only shortened, exactly in the same way as it is displayed in the history, e.g. KeeLoq - KL_, StarLine - SL_, CAME - Came_, and so on.

@RogueMaster RogueMaster reopened this Jul 24, 2024
@RogueMaster RogueMaster merged commit f522427 into RogueMaster:420 Jul 24, 2024
@RogueMaster
Copy link
Owner

Thanks, I have made added all your changes

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.

2 participants