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

Fix things with a few params #16

Closed
wants to merge 5 commits into from

Conversation

Happyrobot33
Copy link

@Happyrobot33 Happyrobot33 commented Apr 11, 2023

Adds a suffix to the package listing
Fixed the ability to set CurrentListingUrl to another URL. I needed to fix this in order for my custom domain to work

closes #15

@github-actions
Copy link

github-actions bot commented Apr 11, 2023

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Happyrobot33
Copy link
Author

Happyrobot33 commented Apr 11, 2023

I have read the CLA Document and I hereby sign the CLA

@Happyrobot33 Happyrobot33 marked this pull request as draft April 12, 2023 17:23
The previous implementation meant it wasnt allowed to be set
@Happyrobot33 Happyrobot33 changed the title make package listing suffix configurable using --Package-Listing-Suffix Fix things with a few params Apr 12, 2023
@Happyrobot33 Happyrobot33 marked this pull request as ready for review April 12, 2023 18:02
@anatawa12
Copy link
Contributor

I think it's better to have configurable name than configurable suffix.

@Spokeek
Copy link

Spokeek commented Nov 8, 2023

Agreed, i would be more into customizing the list name as a whole instead of changing prefixes and suffixes.

@Spokeek
Copy link

Spokeek commented Nov 8, 2023

Also, maybe change the PR name to something like "Make Package Listing Name configurable" ?

@Happyrobot33
Copy link
Author

Closing to make a new PR with just these changes using a new branch on my fork

@github-actions github-actions bot locked and limited conversation to collaborators Oct 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] make the "listing" string configurable
3 participants