Skip to content

feat: Discord OAuth support#1824

Merged
umputun merged 7 commits intoumputun:masterfrom
mgkbadola:discord-oauth
Nov 18, 2024
Merged

feat: Discord OAuth support#1824
umputun merged 7 commits intoumputun:masterfrom
mgkbadola:discord-oauth

Conversation

@mgkbadola
Copy link
Copy Markdown
Contributor

Completes Discord OAuth integration on frontend. Can only be merged when a new version of go-pkgz/auth is released with Discord OAuth backend support (reference).

@paskal
Copy link
Copy Markdown
Collaborator

paskal commented Nov 8, 2024

You can update to latest master of auth package instead and not wait for the version release.

@mgkbadola
Copy link
Copy Markdown
Contributor Author

I've done the same, thanks.

@mgkbadola mgkbadola marked this pull request as ready for review November 10, 2024 07:15
@mgkbadola mgkbadola requested a review from umputun as a code owner November 10, 2024 07:15
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 10, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.24%. Comparing base (3646c4a) to head (d0dc113).
⚠️ Report is 145 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master    #1824    +/-   ##
========================================
  Coverage   64.24%   64.24%            
========================================
  Files         140      140            
  Lines        3230     3230            
  Branches      773      815    +42     
========================================
  Hits         2075     2075            
+ Misses       1151     1038   -113     
- Partials        4      117   +113     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@paskal
Copy link
Copy Markdown
Collaborator

paskal commented Nov 10, 2024

@mgkbadola thanks a lot! Could you please run Remark42 with Discord auth enabled and post a screenshot of how the auth icon looks like and verify that you can log in using that method?

@badolamgk
Copy link
Copy Markdown
Contributor

Hey @paskal
Here are the screenshots from my trial run.

Able to login and comment via Discord as intended.
Screenshot 2024-11-12 at 9 18 29 PM
Screenshot 2024-11-12 at 9 19 40 PM

@umputun umputun merged commit 16ff269 into umputun:master Nov 18, 2024
@paskal
Copy link
Copy Markdown
Collaborator

paskal commented Dec 2, 2024

@umputun interesting case: this PR broke backend tests in master (fixed in #1862), but the latest commit is merge commit which doesn't touch backend so the tests were not run. No idea what to do about it aside from asking the requestor to squash commits locally.

@mgkbadola
Copy link
Copy Markdown
Contributor Author

Let me know how can I help to resolve this

@paskal
Copy link
Copy Markdown
Collaborator

paskal commented Dec 2, 2024

No need to do anything from your side @mgkbadola, it's a fault of our CI setup which doesn't highlight the problem where it should. The failing test is already fixed.

@paskal paskal added the backend label Mar 25, 2026
@paskal paskal added this to the v1.15.0 milestone Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants