-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: add "received at" timestamp to invites #529
Conversation
This will make it possible for the frontend to sort incoming invites.
@@ -90,7 +98,7 @@ | |||
"eslint": "^8.57.0", | |||
"filter-obj": "^6.0.0", | |||
"husky": "^8.0.0", | |||
"iterpal": "^0.3.0", | |||
"iterpal": "^0.4.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Necessary to pull in compact
for tests.
"error", | ||
{ | ||
"varsIgnorePattern": "^_", | ||
"argsIgnorePattern": "^_" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I use varsIgnorePattern
in this PR, but not argsIgnorePattern
. I added it for consistency.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lots of changes to the tests for a small change in the code! Looks ok though. Not sure about the eslint rule - I prefer to ignore on a case-by-case, but ok for this repo since you'll be taking the lead.
tests/invite-api.js
Outdated
@@ -1047,3 +1099,73 @@ function setup() { | |||
encryptionKeys, | |||
} | |||
} | |||
|
|||
/** | |||
* Assert that invites are alike, allowing for some wiggle room around |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add a note about what wiggle room means here, to help a future maintainer who might find this failing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 8229dac.
This will make it possible for the frontend to sort incoming invites, and was requested by Erik.