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

Add New Plugin: Limit Custom Keyword Length (Min & Max Characters) #160

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

suryatanjung
Copy link
Contributor

Limit Custom Keyword Length (Min & Max Characters) - This plugin limits the minimum and maximum number of characters for custom keyword.

Limit Custom Keyword Length (Min & Max Characters) - This plugin limits the minimum and maximum number of characters for custom keyword.
Copy link
Contributor Author

@suryatanjung suryatanjung left a comment

Choose a reason for hiding this comment

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

Add New Plugin: Limit Custom Keyword Length (Min & Max Characters)

README.md Show resolved Hide resolved
Copy link
Member

@dgw dgw left a comment

Choose a reason for hiding this comment

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

Future versions might support configuring the limits in the admin panel? But there's no reason not to accept this new plugin link now.

@suryatanjung
Copy link
Contributor Author

thank you hope this useful for future versions

Copy link
Contributor Author

@suryatanjung suryatanjung left a comment

Choose a reason for hiding this comment

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

sorry i mistakenly closed this patch

@suryatanjung suryatanjung reopened this Jan 26, 2024
@suryatanjung suryatanjung requested a review from dgw January 26, 2024 11:02
Copy link
Member

@dgw dgw left a comment

Choose a reason for hiding this comment

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

I had already approved this. Please just wait for another maintainer to take a look, thanks.

@LeoColomb LeoColomb merged commit c149470 into YOURLS:main Mar 4, 2024
3 checks passed
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