Skip to content

Fix: create time#1250

Merged
Weiko merged 4 commits intotwentyhq:mainfrom
sunilk4u:fix/create-time
Aug 23, 2023
Merged

Fix: create time#1250
Weiko merged 4 commits intotwentyhq:mainfrom
sunilk4u:fix/create-time

Conversation

@sunilk4u
Copy link
Copy Markdown
Contributor

Fixes #1240
image

@ergomake
Copy link
Copy Markdown

ergomake bot commented Aug 17, 2023

Hi 👋

Here's a preview environment 🚀

https://front-twentyhq-twenty-1250.env.ergomake.link

Environment Summary 📑

Container Source URL
front Dockerfile https://front-twentyhq-twenty-1250.env.ergomake.link
server Dockerfile https://server-twentyhq-twenty-1250.env.ergomake.link
postgres Dockerfile [not exposed - internal service]

Here are your environment's logs.

For questions or comments, join Discord.

Click here to disable Ergomake.

Copy link
Copy Markdown
Member

@Weiko Weiko left a comment

Choose a reason for hiding this comment

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

@sunilk4u The test was not using the new addSuffix parameter so the assertion was failing, I pushed the change. Looks good, we can merge, thanks a lot!

@Weiko Weiko merged commit 587c746 into twentyhq:main Aug 23, 2023
@sunilk4u
Copy link
Copy Markdown
Contributor Author

@sunilk4u The test was not using the new addSuffix parameter so the assertion was failing, I pushed the change. Looks good, we can merge, thanks a lot!

Thanks @Weiko, I forgot about this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change the timestamp values for "less than a minute"

2 participants