Skip to content
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

58 feat discord bot to sync role between GitHub and discord server #70

Merged

Conversation

42atomys
Copy link
Owner

@42atomys 42atomys commented Apr 20, 2022

Describe the pull request
Add a feature to invite sponsors to the discord before opening as well as its discord bot to sync the roles with their Github / Third party sponsors role

Checklist

  • I have linked the relative issue to this pull request
  • I have made the modifications or added tests related to my PR
  • I have added/updated the documentation for my RP
  • I put my PR in Ready for Review only when all the checklist is checked

Breaking changes ?
yes

Additional context

Resolves #66, Resolves #67, Resolves #68, Resolves #69

* feat: Implement the JWTKS Service

* docs: Add doc to start JWTKS Service without DEVENV

* feat: Secure the GRPC server with certificates

* fix: Install protoc compiler dependencies on CI

* fix: Log of JWTKS certificates cause tests crashes
@42atomys 42atomys added this to the 3.0 Beta Release 👷 milestone Apr 20, 2022
@42atomys 42atomys self-assigned this Apr 20, 2022
@42atomys 42atomys linked an issue Apr 20, 2022 that may be closed by this pull request
11 tasks
Copy link
Owner Author

@42atomys 42atomys left a comment

Choose a reason for hiding this comment

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

Rework to do before merging

internal/api/authentication.go Show resolved Hide resolved
internal/api/authentication.go Outdated Show resolved Hide resolved
internal/jwtks/grpc.go Outdated Show resolved Hide resolved
web/ui/.vscode/settings.json Outdated Show resolved Hide resolved
web/ui/package.json Outdated Show resolved Hide resolved
web/ui/src/grpc/jwtks/index.ts Outdated Show resolved Hide resolved
web/ui/src/lib/jwt.ts Outdated Show resolved Hide resolved
web/ui/src/pages/api/auth/[...nextauth].ts Outdated Show resolved Hide resolved
web/ui/src/pages/api/auth/[...nextauth].ts Outdated Show resolved Hide resolved
web/ui/src/pages/discord/_middleware.ts Outdated Show resolved Hide resolved
Copy link
Owner Author

@42atomys 42atomys left a comment

Choose a reason for hiding this comment

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

LGTM

@42atomys 42atomys merged commit 37715ce into main Apr 20, 2022
@42atomys 42atomys deleted the 58-feat-discord-bot-to-sync-role-between-github-and-discord-server branch April 20, 2022 22:52
42atomys added a commit that referenced this pull request Apr 21, 2022
* fix(types): Fix wrong types

* feat: Add Discord OAuth2 processs

* unknown: Just push

* feat: Implement the JWTKS Service (#65)

* feat: Implement the JWTKS Service

* docs: Add doc to start JWTKS Service without DEVENV

* feat: Secure the GRPC server with certificates

* fix: Install protoc compiler dependencies on CI

* fix: Log of JWTKS certificates cause tests crashes

* docs: Add functions documentations for the JWTKS Service

* feat: Adding the Discord homepage with steps to link 42 / Discord and Github

* feat: Use Feature concept to know if an User is a sponsors or not for Discord page

* feat: Join Discord button will now invite the user without create invite link

* clean: Remove cached files

* chore: Add TODO comments

* docs: Add EnvVar to env.example

* chore: Follow the lint directive

* fix: Remove useless checks

* feat: Upgrade to React 18

* chore: Do review

* chore: Do review on package.json

* fix: Make JWKS endpoitn configurable

* feat: Make frontend endpoints configurable with config file

* docs: Add documentation

* docs: Add documentation redirection for config

* fix: Install protoc for frontend tests

* chore: Ignore generated files fron prettier
42atomys added a commit that referenced this pull request Nov 12, 2022
* fix(types): Fix wrong types

* feat: Add Discord OAuth2 processs

* unknown: Just push

* feat: Implement the JWTKS Service (#65)

* feat: Implement the JWTKS Service

* docs: Add doc to start JWTKS Service without DEVENV

* feat: Secure the GRPC server with certificates

* fix: Install protoc compiler dependencies on CI

* fix: Log of JWTKS certificates cause tests crashes

* docs: Add functions documentations for the JWTKS Service

* feat: Adding the Discord homepage with steps to link 42 / Discord and Github

* feat: Use Feature concept to know if an User is a sponsors or not for Discord page

* feat: Join Discord button will now invite the user without create invite link

* clean: Remove cached files

* chore: Add TODO comments

* docs: Add EnvVar to env.example

* chore: Follow the lint directive

* fix: Remove useless checks

* feat: Upgrade to React 18

* chore: Do review

* chore: Do review on package.json

* fix: Make JWKS endpoitn configurable

* feat: Make frontend endpoints configurable with config file

* docs: Add documentation

* docs: Add documentation redirection for config

* fix: Install protoc for frontend tests

* chore: Ignore generated files fron prettier
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment