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

Amato App Submission #729

Closed
wants to merge 13 commits into from
Closed

Amato App Submission #729

wants to merge 13 commits into from

Conversation

ziadgit
Copy link

@ziadgit ziadgit commented Aug 17, 2023

App Submission

App name

Amato Timer

256x256 SVG icon

(Submit an icon with no rounded corners as it will be dynamically rounded with CSS. GitHub doesn't allow uploading SVGs directly, so please upload your icon to an alternate service, like https://svgur.com, and paste the link below.)

https://svgshare.com/i/wdB.svg

Gallery images

(Upload 3 to 5 high-quality gallery images (1440x900px) of your app in PNG format, or just upload 3 to 5 screenshots of your app and we'll help you design the gallery images.)

1
2
3

I have tested my app on:

amato/umbrel-app.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Outdated Show resolved Hide resolved
@ParthJadhav
Copy link
Contributor

Also @ziadgit , Can you add direct link to SVG to the description:

https://svgshare.com/i/wdB.svg

Copy link
Contributor

@ParthJadhav ParthJadhav left a comment

Choose a reason for hiding this comment

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

Tested on Umbrel-Dev server, Works as expected!

feel free to make the changes listed above 👆

Thanks for the contribution ❤️

ziadgit and others added 2 commits September 25, 2023 05:28
edit release notes to null

Co-authored-by: Parth Jadhav <[email protected]>
add submitter and submission data
amato/umbrel-app.yml Outdated Show resolved Hide resolved
remove duplicate/erroneous releaseNotes field
@ParthJadhav
Copy link
Contributor

Looks good to me @nmfretz

Copy link
Contributor

@nmfretz nmfretz left a comment

Choose a reason for hiding this comment

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

Thanks for submitting Amato Timer @ziadgit! I have tested and left a review below.

Are you the developer of this app? I noticed a couple things that should be clarified:

  • You have submitted this app as Amato Timer, but I see that the UI calls the app Pomodoro Timer and the docker image is tagged as berkeleyengineer/pomoro. I think we should try to align some/all of these so that it is clear what the app is and where it is coming from. What are your thoughts on this?
  • You have listed Bitparasol as the developer and linked to the bitparasol website. Is this referring to you and your site? I did not see anything related to Amato timer on this site. If you are the developer, then we should make developer and submitter the same, and link to something that describes Amato Timer or shows the source code.

amato/docker-compose.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Show resolved Hide resolved
amato/umbrel-app.yml Show resolved Hide resolved
amato/umbrel-app.yml Outdated Show resolved Hide resolved
amato/umbrel-app.yml Show resolved Hide resolved
@ziadgit
Copy link
Author

ziadgit commented Oct 12, 2023

I believe I have addressed all the branding concerns in the UI and the schema and linked to appropriate bitparasol pages. I've also updated the image to reference the specific sha of the latest image. If after testing this is approved I can update the screenshots.

@nmfretz
Copy link
Contributor

nmfretz commented Oct 17, 2023

Excellent, thanks for addressing everything @ziadgit. We can help convert screenshots into gallery images for you and also help format your icon svg. I'll post them here soon for your approval and then we can send this to the app store!

FYI, I changed the submitter to "bitparasol" so that it shows up nicely in the UI. Otherwise it ends up looking like this in the app store.

image

When someone clicks the "bitparasol" hyperlink next to "Submitted by" it brings them to this PR showing you as submitter.

@highghlow
Copy link
Contributor

Hello, @nmfretz , what's keeping this PR from merging?

@nmfretz
Copy link
Contributor

nmfretz commented Jun 24, 2024

Thanks for pinging me here @highghlow.

@ziadgit, I've just reviewed this again. Is Amato Timer a personal project for learning to code and package an app for umbrelOS? I ask becasue a tomato timer is a common tutorial app for programming. If so, we're hesitant to add this to the official app store because there may not be a large incentive for you (or other developers) to keep maintaining the app.

It is awesome that you're building something for umbrelOS. If you're trying to just start out experimenting with a simple app, then there are a couple of other options that might be better at this early stage in Amato Timer's development:

  1. Create a Community App Store: https://github.com/getumbrel/umbrel-community-app-store

    You and other users can add this app store to umbrelOS through the UI. This allows you to distribute and test the app while you continue developing it. It's a good way to gather feedback and improve your app with real user input.

    image
  2. Use the Portainer app on umbrelOS: https://apps.umbrel.com/app/portainer

    Portainer allows you to run any compose file, so you can very quickly set up and test anything through it.

Let me know if either of those options would work for you.

@ziadgit
Copy link
Author

ziadgit commented Jun 24, 2024 via email

@nmfretz
Copy link
Contributor

nmfretz commented Sep 12, 2024

Understood, thanks for that explanation @ziadgit!

If you're comfortable with this, let's do the following:

I see your app is primarily built with a basic HTML structure and minimal custom styling, relying on the browser’s default rendering of HTML elements like <h1> and <button>. Would you consider adding custom styling (through vanilla CSS or a component library or framework) so that it can stand the test of time against other Tomato Timer apps out there even if you move on to other things and end up never updating Amato again?

@nmfretz
Copy link
Contributor

nmfretz commented Sep 18, 2024

@ziadgit I am going to close this PR for now, but please reopen when/if you end up working on Amato again.

@nmfretz nmfretz closed this Sep 18, 2024
@ziadgit
Copy link
Author

ziadgit commented Sep 18, 2024

Hi Nmfretz,

I'm working on the changes, but have been moving the bitparasol domain this week.

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