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

Create wishlist.xml #518

Closed
wants to merge 1 commit into from
Closed

Create wishlist.xml #518

wants to merge 1 commit into from

Conversation

ajb3932
Copy link

@ajb3932 ajb3932 commented Sep 17, 2024

A great tool for creating a present wishlist.

Created by Wingysam
Converted by ajb3932</Overview>
<Category>Productivity: Tools:</Category>
<WebUI>http://[IP]:[PORT:8023]/</WebUI>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<WebUI>http://[IP]:[PORT:8023]/</WebUI>
<WebUI>http://[IP]:[PORT:80]/</WebUI>

Fallback needs to be the internal port on the container, in this case 80.

<DonateText/>
<DonateLink/>
<Requires/>
<Config Name="Web UI" Target="80" Default="80" Mode="tcp" Description="" Type="Port" Display="always" Required="true" Mask="false">8023</Config>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Config Name="Web UI" Target="80" Default="80" Mode="tcp" Description="" Type="Port" Display="always" Required="true" Mask="false">8023</Config>
<Config Name="Web UI" Target="80" Default="8023" Mode="tcp" Description="" Type="Port" Display="always" Required="true" Mask="false">8023</Config>

Default in XML should match the pre-filled value.

<DonateLink/>
<Requires/>
<Config Name="Web UI" Target="80" Default="80" Mode="tcp" Description="" Type="Port" Display="always" Required="true" Mask="false">8023</Config>
<Config Name="List Mode" Target="SINGLE_LIST" Default="false" Mode="" Description="Single list mode&#13;&#10; (for weddings, birthdays, etc. only the admin account's list is accessible)&#13;&#10; Set to 'true' to enable" Type="Variable" Display="always" Required="false" Mask="false">true</Config>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Config Name="List Mode" Target="SINGLE_LIST" Default="false" Mode="" Description="Single list mode&#13;&#10; (for weddings, birthdays, etc. only the admin account's list is accessible)&#13;&#10; Set to 'true' to enable" Type="Variable" Display="always" Required="false" Mask="false">true</Config>
<Config Name="Enable Single List Mode" Target="SINGLE_LIST" Default="false|true" Mode="" Description="Single list mode (for weddings, birthdays, etc., only the admin account's list is accessible)." Type="Variable" Display="always" Required="true" Mask="false"/>

<Config Name="Web UI" Target="80" Default="80" Mode="tcp" Description="" Type="Port" Display="always" Required="true" Mask="false">8023</Config>
<Config Name="List Mode" Target="SINGLE_LIST" Default="false" Mode="" Description="Single list mode&#13;&#10; (for weddings, birthdays, etc. only the admin account's list is accessible)&#13;&#10; Set to 'true' to enable" Type="Variable" Display="always" Required="false" Mask="false">true</Config>
<Config Name="Amazon Smile" Target="SMILE" Default="true" Mode="" Description="Amazon Smile, set to 'false' to disable." Type="Variable" Display="always" Required="false" Mask="false">true</Config>
<Config Name="Data Dir" Target="/data" Default="/mnt/user/appdata/wishlist" Mode="rw" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/appdata/wishlist</Config>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Config Name="Data Dir" Target="/data" Default="/mnt/user/appdata/wishlist" Mode="rw" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/appdata/wishlist</Config>
<Config Name="Data Directory" Target="/data" Default="/mnt/user/appdata/wishlist" Mode="rw" Description="" Type="Path" Display="advanced" Required="true" Mask="false">/mnt/user/appdata/wishlist</Config>

<Config Name="List Mode" Target="SINGLE_LIST" Default="false" Mode="" Description="Single list mode&#13;&#10; (for weddings, birthdays, etc. only the admin account's list is accessible)&#13;&#10; Set to 'true' to enable" Type="Variable" Display="always" Required="false" Mask="false">true</Config>
<Config Name="Amazon Smile" Target="SMILE" Default="true" Mode="" Description="Amazon Smile, set to 'false' to disable." Type="Variable" Display="always" Required="false" Mask="false">true</Config>
<Config Name="Data Dir" Target="/data" Default="/mnt/user/appdata/wishlist" Mode="rw" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/appdata/wishlist</Config>
<Config Name="Public Lists" Target="LISTS_PUBLIC" Default="false" Mode="" Description="" Type="Variable" Display="always" Required="false" Mask="false">true</Config>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Config Name="Public Lists" Target="LISTS_PUBLIC" Default="false" Mode="" Description="" Type="Variable" Display="always" Required="false" Mask="false">true</Config>
<Config Name="Enable Public Lists" Target="LISTS_PUBLIC" Default="false|true" Mode="" Description="" Type="Variable" Display="always" Required="true" Mask="false"/>

templates/wishlist.xml Show resolved Hide resolved
@@ -0,0 +1,31 @@
<?xml version="1.0"?>
<Container version="2">
<Name>Wishlist</Name>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Name>Wishlist</Name>
<Name>Christmas Community</Name>

Keep the name of the upstream app.

<Network>bridge</Network>
<Shell>sh</Shell>
<Privileged>false</Privileged>
<Support>https://hub.docker.com/r/wingysam/christmas-community/</Support>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Support>https://hub.docker.com/r/wingysam/christmas-community/</Support>
<Support>https://github.com/Wingysam/Christmas-Community/issues</Support>

Point it to the GitHub issues page instead.

<Shell>sh</Shell>
<Privileged>false</Privileged>
<Support>https://hub.docker.com/r/wingysam/christmas-community/</Support>
<Project>https://github.com/Wingysam/Christmas-Community/tree/master</Project>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Project>https://github.com/Wingysam/Christmas-Community/tree/master</Project>
<Project>https://github.com/Wingysam/Christmas-Community</Project>

Remove specific branch in link.

Comment on lines +11 to +14
<Overview>To create a simple place for your entire family to use to find gifts that people want, and to avoid double-gifting.&#xD;
https://github.com/Wingysam/Christmas-Community/tree/master&#xD;
Created by Wingysam&#xD;
Converted by ajb3932</Overview>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<Overview>To create a simple place for your entire family to use to find gifts that people want, and to avoid double-gifting.&#xD;
https://github.com/Wingysam/Christmas-Community/tree/master&#xD;
Created by Wingysam&#xD;
Converted by ajb3932</Overview>
<Overview>A web app for your family's Christmas shopping. Create a simple place for your entire family to use to find gifts that people want, and to avoid double-gifting.</Overview>

@ajb3932
Copy link
Author

ajb3932 commented Sep 18, 2024

Hey sorry I'm kinda new to PRs. I would like to add the devs donate link into <DonateText>Donate</DonateText> & <DonateLink>https://ko-fi.com/wingy</DonateLink>. Is this possible in the current PR?

@Roxedus
Copy link
Member

Roxedus commented Sep 18, 2024

Yes, you can add that, as well as actually resolving the review items.

@ajb3932
Copy link
Author

ajb3932 commented Sep 24, 2024

I'm working on my own wishlist app so will make my own template. Sorry to waste your time. 😊

@ajb3932 ajb3932 closed this Sep 24, 2024
@ajb3932 ajb3932 deleted the patch-1 branch September 24, 2024 13:29
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.

3 participants