+
Skip to content

Conversation

lysnikolaou
Copy link
Member

What does this PR do?

  • Main bot structure in bot directory
  • Integration with database to record users that have already accepted the COC.
  • Automatic migrations with alembic

How to test changes / Special notes to the reviewer

  • Already tested all the changes in dummy server

- Main bot structure in `bot` directory
- Integration with database to record users that have already accepted
  the COC.
- Automatic migrations with alembic
@lysnikolaou
Copy link
Member Author

I will follow-up in another PR with the following:

  • CI/CD to run ruff, mypy etc.
  • Deployment setup to deploy to DigitalOcean with a postgres DB.

@lysnikolaou lysnikolaou changed the title Implement Discord bot for automatic bot assignment Implement Discord bot for automatic role assignment Mar 16, 2025
@lysnikolaou lysnikolaou force-pushed the feature/automic-role-assignment branch from f6fa6f8 to 7a7ec20 Compare March 16, 2025 19:06
Copy link
Contributor

@thepetk thepetk left a comment

Choose a reason for hiding this comment

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

Γενικα lgtm, αυτο που σκεφτομαι τωρα με το bot ειναι αν εχει νοημα να κανουμε ενα μονιμο discord server σαν dev περιβαλλον για να τεσταρουμε τις αλλαγες πριν τις περασουμε στο pygreece? Ποιο πολυ απο την αποψη οτι θα θελει να ασχοληθει και καποιο ατομο χωρις εμπειρια, να μην χρειαζεται να στησει κατι ιδιαιτερο. Τι λετε;

@gzisopoulos
Copy link

Γενικα lgtm, αυτο που σκεφτομαι τωρα με το bot ειναι αν εχει νοημα να κανουμε ενα μονιμο discord server σαν dev περιβαλλον για να τεσταρουμε τις αλλαγες πριν τις περασουμε στο pygreece? Ποιο πολυ απο την αποψη οτι θα θελει να ασχοληθει και καποιο ατομο χωρις εμπειρια, να μην χρειαζεται να στησει κατι ιδιαιτερο. Τι λετε;

 Θα συμφωνησω. Τρωει λιγακι χρονο το να το σεταρει κάποιος. Ας έχουμε ένα μονο για αυτο το σκοπο

@lysnikolaou
Copy link
Member Author

Γενικα lgtm, αυτο που σκεφτομαι τωρα με το bot ειναι αν εχει νοημα να κανουμε ενα μονιμο discord server σαν dev περιβαλλον για να τεσταρουμε τις αλλαγες πριν τις περασουμε στο pygreece? Ποιο πολυ απο την αποψη οτι θα θελει να ασχοληθει και καποιο ατομο χωρις εμπειρια, να μην χρειαζεται να στησει κατι ιδιαιτερο. Τι λετε;

Συμφωνω απολυτα. Εχω εναν τεστ σερβερ, στον οποιον εκανα ενα minial setup σαν αυτο που εχουμε στο PyGreece. Να χρησιμοποιησουμε αυτον;

Copy link
Contributor

@thepetk thepetk left a comment

Choose a reason for hiding this comment

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

Μια χαρα μου φαινεται, οπως γραφω και παραπανω μπορουμε να το τεσταρουμε και σε καποιον μονιμο dummy/dev server (οπως και καθε νεα αλλαγη).

Για το κομματι του deployment (see σχολιο στον σερβερ) νομιζω θα μπορουσαμε να παμε στη λογικη ενος droplet θα τρεχει το bot, αρα ισως θα πρεπει να το βαλουμε σε container (δλδ βαζουμε και ενα απλο dockerfile).

Ωστοσο, φανταζομαι μπορει να υπαρχουν εναλλακτικες που εχουμε σκεφτει - για αυτό και η ερώτηση κυρίως από περιεργια <3

@lysnikolaou lysnikolaou merged commit 5c92fa9 into pygreece:main Mar 16, 2025
@lysnikolaou lysnikolaou deleted the feature/automic-role-assignment branch March 16, 2025 21:00
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.

3 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载