-
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
Bluesky post with url link #15
Conversation
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.
🚀 LGTM (modulo the environment variables in the docker-compose file that seem useless)
@@ -19,6 +19,10 @@ services: | |||
- .env | |||
ports: | |||
- "5432:5432" | |||
environment: |
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.
polish: explicitly declare these variables is not needed because the environment variables are already defined in the .env
file.
@@ -4,6 +4,7 @@ | |||
"scripts": { | |||
"build": "nuxt build", | |||
"dev": "nuxt dev", | |||
"test": "vitest", |
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.
praise: THANK YOU !!! ❤️
try { | ||
const record: any = {}; | ||
record.text = message.text; | ||
record.facets = buildUrlFacets(message.text); |
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.
praise: Very pragmatic. Very cool.
No description provided.