Skip to content

Morse Master and Moon Phases #763

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

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

w84death
Copy link

Application Submission

image

The Morse Master is an interactive application for Flipper Zero that helps you learn and practice Morse code. The app offers two main modes: a Learning Mode to familiarize yourself with Morse code patterns and a Practice Mode to test your skills.

Author Checklist (Fill this out)

  • I've read the contribution guidelines and my PR follows them
  • I own the code I'm submitting or have code owner's permission to submit it
  • I have validated the manifest file(s) with python3 tools/bundle.py --nolint applications/CATEGORY/APPID/manifest.yml bundle.zip

Reviewer Checklist (Don't fill this out)

  • Bundle is valid
  • There are no obvious issues with the source code
  • I've ran this application and verified its functionality

@w84death
Copy link
Author

I hope I did made this correctly. As the Moon Phases app PR was having some problems, here is the second application.

Morse Master is way more useful, feature rich and actually helpful. I would like to have this one in the catalog over the last simple one. Worse case scenario I can discard everything and do new fork and add just Morse Master to the catalog.

As good I am with git, the public PR's are new to me. I got a little bit confused with having two on the same codebase (and two branches).

@doomwastaken
Copy link
Member

Nvm previous comment, I was being silly, let me try in my own PR

@w84death
Copy link
Author

OK. Too bad I did not see the previous one ;)

@doomwastaken doomwastaken self-assigned this Jun 2, 2025
@doomwastaken
Copy link
Member

Actually you can just update this PR alone, no need to update previous one. I will close it. Update both manifest.yaml -> manifest.yml and bump commit hash in both repos. Thank you for your patience

@doomwastaken doomwastaken added the QA pending To be reviewed by QA label Jun 11, 2025
@doomwastaken
Copy link
Member

Fixed extensions myself, next fix is on the side of your repo.
https://github.com/w84death/moon-phases/blob/main/application.fam (and second app too)
appid="p1x_moon_phases",
must match one that is in the PR
Error: 09:08:11.720 [E] App ID 'p1x_morse_master' in manifest does not match app ID 'morse_master' from path '/home/runner/work/flipper-application-catalog/flipper-application-catalog/applications/Media/morse_master/manifest.yml'

@doomwastaken doomwastaken added fixes needed Unresolved issues with the app requiring maintainer's attention and removed QA pending To be reviewed by QA labels Jun 13, 2025
@w84death w84death changed the title Morse Master Morse Master and Moon Phases Jun 23, 2025
@doomwastaken
Copy link
Member

doomwastaken commented Jun 24, 2025

Header is missing in the build
Error: /tmp/tmpw56t_31c/code/p1x_moon_phases.c:15:10: fatal error: p1x_moon_phases_icons.h: No such file or directory

same for other app

All of this can be validated here via this part

I have validated the manifest file(s) with python3 tools/bundle.py --nolint applications/CATEGORY/APPID/manifest.yml bundle.zip

@w84death
Copy link
Author

w84death commented Jun 26, 2025

That is strange as I got those files before. I'm doing those new pushes from other computer. I will check the old one and restore them or create new one. It's not in gitignore so they should be in the repo. But this explains one bug report that icons was missing.

EDIT: this file is created automatically but not saved. I will re-read the manual and make it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app (new) fixes needed Unresolved issues with the app requiring maintainer's attention
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants