Skip to content

Conversation

@dipamsen
Copy link
Member

Description

This PR generalises the slack.rs file to take any message and send it to a slack webhook. This allows us to also add slack notifications for when a library archive is imported into the application.

  • I have updated the relevant documentation

@vercel
Copy link

vercel bot commented Aug 12, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
iqps Ready Ready Preview Comment Aug 16, 2025 2:37pm

@dipamsen
Copy link
Member Author

Maybe i can also send the corresponding logfile to slack 👀

@harshkhandeparkar
Copy link
Member

Maybe i can also send the corresponding logfile to slack 👀

What does the logfile have?

Copy link
Member

@harshkhandeparkar harshkhandeparkar left a comment

Choose a reason for hiding this comment

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

Can this be merged independently of #125?

@dipamsen
Copy link
Member Author

Can this be merged independently of #125?

Yes

What does the logfile have?

Stuff like x paper was skipped because already present, etc

@harshkhandeparkar
Copy link
Member

What does the logfile have?

Stuff like x paper was skipped because already present, etc

Then it would be nice to send it too. Where is it stored normally? Would be good to store it in the same log directory as the main backend (but in a separate file).

@dipamsen
Copy link
Member Author

dipamsen commented Aug 12, 2025

Where is it stored normally?

Its actually just stored inside /backend for now, but can be changed

@harshkhandeparkar
Copy link
Member

Where is it stored normally?

Its actually just stored inside /backend for now, but can be changed

It would be good to store in the logdir used by the main backend.

@harshkhandeparkar harshkhandeparkar merged commit b7e9988 into metakgp:main Aug 16, 2025
3 checks 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.

2 participants