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 Send Timestamps #149

Merged
merged 2 commits into from
Jan 22, 2025
Merged

Create Send Timestamps #149

merged 2 commits into from
Jan 22, 2025

Conversation

uwx
Copy link
Contributor

@uwx uwx commented Jan 22, 2025

Adds a .time command to send timestamps written in natural language.
For instance: .send tomorrow at 2pm will send a message with a timestamp pointing to "tomorrow at 2pm" local time.

To send in alternative formats, use one of: .timeshortdate, .timelongdate, .timelongdateshorttime, .timefull, .timeshorttime, .timelongtime, .timerelative. Defaults to .timerelative.

Powered by Microsoft.Recognizers.Text.

@NotNite
Copy link
Member

NotNite commented Jan 22, 2025

No actual code review yet (I know you're testing if it even builds), but one small nitpick is that I think this might fit better as an actual slash command?

@uwx uwx marked this pull request as ready for review January 22, 2025 21:16
@uwx
Copy link
Contributor Author

uwx commented Jan 22, 2025

.asar appears to have built correctly!

@NotNite
Copy link
Member

NotNite commented Jan 22, 2025

Another small thing I noticed is that you refer to ".send" but that doesn't actually exist - seems to be a typo.

@uwx
Copy link
Contributor Author

uwx commented Jan 22, 2025

No actual code review yet (I know you're testing if it even builds), but one small nitpick is that I think this might fit better as an actual slash command?

Like a user app?

@Cynosphere Cynosphere marked this pull request as draft January 22, 2025 21:23
@uwx
Copy link
Contributor Author

uwx commented Jan 22, 2025

Non legacy command is real

@uwx uwx marked this pull request as ready for review January 22, 2025 22:05
@Cynosphere Cynosphere merged commit dfdefaa into moonlight-mod:main Jan 22, 2025
1 check 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