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

Poor "Time Formatting" in README #6

Open
accessfletch opened this issue Jan 5, 2021 · 5 comments
Open

Poor "Time Formatting" in README #6

accessfletch opened this issue Jan 5, 2021 · 5 comments

Comments

@accessfletch
Copy link

accessfletch commented Jan 5, 2021

I'm trying to set a reminder, but am having trouble with the time formatting.

As the README says, the app is using agenda.js and its time formatting rules... but as a non-programmer, I can't seem to tell what those rules are.

If I enter /remind me to test at 12:14 pm (or /remind me to test at 12:14pm, for that matter), I get the "fail to understand" message.

What would be the proper format for a specific time... or a specific date?

@shoei101
Copy link

shoei101 commented Jan 6, 2021

from what i've seen, it's kind of like a timer, so you have to say "/remind me to test in 10 minutes" or "in 2 hours" I'm hoping with the future updates we will be able to just put a time like an alarm.

With that all said, i'm having issues getting the reminders to even show...

@fakepop
Copy link
Contributor

fakepop commented Feb 11, 2021

As per agenda.js, it uses Human Interval.

I'd be great if it indeed supported dates / times, and also relative time such as "tomorrow at 8am" or "Friday at 3pm".

@robertfromont
Copy link

It appears to use a fixed template:
/remind who to what in when

If you put
/remind me to test in at 12:14 pm
...the command is accepted.

@mjovanovic0 it would be great if at could replace in in the command template. Looks like that change could be added at/after

const lastIn: number = fullCommand.lastIndexOf(' in ');

Maybe something like:

...
        const lastIn: number = fullCommand.lastIndexOf(' in ');
        const lastAt: number = fullCommand.lastIndexOf(' at ');
        const when: string = fullCommand.substring((lastIn>=0?lastIn:lastAt) + 4);
        
        if (lastIn <= 0 && lastAt <= 0) {
...

(and tweak the response message to use 'at' when the user does)

@mjovanovic0
Copy link
Member

Hi all,

I just submitted new version of app.
This version includes in and at option for selector of time component.

Will try to implement particular time (like 12:14 pm, or 1.1.2022) reference in the following update.

@shoei101
Copy link

shoei101 commented Apr 8, 2021

Is this project dead?

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

No branches or pull requests

5 participants