Add io.vikunja.Vikunja.appdata.xml and io.vikunja.Vikunja.desktop #162

Closed
RaptaG wants to merge 3 commits from (deleted):main into main
First-time contributor
Per https://github.com/flathub/flathub/pull/4319#discussion_r1258602996 Signed-off-by: RaptaG <sir-raptag@tutamail.com>
RaptaG added 1 commit 2023-07-10 17:38:46 +00:00
Owner

Can you check how we can tell electron builder to use this? IIRC they already to this?

Check out https://www.electron.build/configuration/linux with the desktop option iirc

Can you check how we can tell electron builder to use this? IIRC they already to this? Check out https://www.electron.build/configuration/linux with the `desktop` option iirc
Author
First-time contributor

target = flatpak is a start I think

`target = flatpak` is a start I think
Owner

what do we get if we add this? IIRC it will only produce a bundle installable with the flatpack cli but nothing we can use to add this to flathub (as they build everything themselves)

what do we get if we add this? IIRC it will only produce a bundle installable with the `flatpack` cli but nothing we can use to add this to flathub (as they build everything themselves)
Author
First-time contributor

🤷‍♂️

:man_shrugging:
Author
First-time contributor

I'll work on this later

I'll work on this later
RaptaG added 1 commit 2023-07-11 14:26:39 +00:00
46b5eecc63
Add a .desktop file
Signed-off-by: RaptaG <sir-raptag@tutamail.com>
RaptaG changed title from Add io.vikunja.Vikunja.appdata.xml to Add io.vikunja.Vikunja.appdata.xml and io.vikunja.Vikunja.desktop 2023-07-11 14:27:21 +00:00
Author
First-time contributor

@konrad the .tar.gz does not have an icon.png inside of it. What should I do?

@konrad the `.tar.gz` does not have an icon.png inside of it. What should I do?
konrad reviewed 2023-07-11 16:35:19 +00:00
@ -0,0 +6,4 @@
Category=Utility;Accessories;
Keywords=todo;task;
Icon=
Exec=flatpak run io.vikunja.Vikunja

If this is specific to flatpack, we should not include it here but in the flathub build repo.

If this is specific to flatpack, we should not include it here but in the flathub build repo.
Author
First-time contributor

If this is specific to flatpack, we should not include it here but in the flathub build repo.

We can only use resource available inside the archive, all processes are done offline

> If this is specific to flatpack, we should not include it here but in the flathub build repo. We can only use resource available inside the archive, all processes are done offline

But we're including the sh wrapper as well? Couldn't we add the .desktop file in a similar manner?

But we're including the sh wrapper as well? Couldn't we add the `.desktop` file in a similar manner?
Owner

@konrad the .tar.gz does not have an icon.png inside of it. What should I do?

mhh I think the icon is bundled in the .asar file but there's no way we'll get it out there during the flatpack build process. Probably best to add it seperately.

> @konrad the `.tar.gz` does not have an icon.png inside of it. What should I do? mhh I *think* the icon is bundled in the `.asar` file but there's no way we'll get it out there during the flatpack build process. Probably best to add it seperately.
Author
First-time contributor

We might include these inside the Flathub repo (I hope)

We might include these inside the Flathub repo (I hope)
RaptaG added 1 commit 2023-07-11 17:58:04 +00:00
1517cec335
Add the icon
Signed-off-by: RaptaG <sir-raptag@tutamail.com>
Author
First-time contributor

Pushed on the Flathub repo 🎉 which now works!!

Pushed on the Flathub repo :tada: which now works!!
RaptaG closed this pull request 2023-07-12 11:29:51 +00:00
Owner

Wow, thanks a lot!

Wow, thanks a lot!
This repo is archived. You cannot comment on pull requests.
No reviewers
No Milestone
No Assignees
2 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/desktop#162
No description provided.