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

feat: mattermost notifier #436

Merged
merged 12 commits into from
Jun 28, 2024
Merged

feat: mattermost notifier #436

merged 12 commits into from
Jun 28, 2024

Conversation

fuuuzz
Copy link
Contributor

@fuuuzz fuuuzz commented May 31, 2024

No description provided.

@fuuuzz fuuuzz changed the title Init mm nitifier feat: mattermost notifier May 31, 2024
@mmarchois mmarchois force-pushed the feat-mm-notifications branch from 36f4e9b to cd4021f Compare May 31, 2024 12:35
.env Outdated
@@ -9,3 +9,4 @@ SESSION_SECRET=changeme
SESSION_COOKIE_SECURE=false
FLUENT_PATH=src/translations/fr-FR.ftl
FLUENT_LOCALE=fr-FR
PERMACOOP_BASE_URL=https://permacoop.fairness.com
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
PERMACOOP_BASE_URL=https://permacoop.fairness.com
PERMACOOP_BASE_URL=https://permacoop.fairness.coop

@fuuuzz fuuuzz force-pushed the feat-mm-notifications branch from dab9e0b to 93dc3d0 Compare June 14, 2024 09:31
Copy link

codecov bot commented Jun 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.32%. Comparing base (26b471a) to head (46c6ecf).

Current head 46c6ecf differs from pull request most recent head db5e30c

Please upload reports for the commit db5e30c to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #436      +/-   ##
==========================================
- Coverage   91.12%   90.32%   -0.80%     
==========================================
  Files         185      189       +4     
  Lines        2130     2213      +83     
  Branches      153      158       +5     
==========================================
+ Hits         1941     1999      +58     
- Misses        182      205      +23     
- Partials        7        9       +2     

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

@fuuuzz fuuuzz force-pushed the feat-mm-notifications branch from 3315ff0 to 47beb6c Compare June 14, 2024 09:53
@fuuuzz fuuuzz force-pushed the feat-mm-notifications branch from 659e436 to 46c6ecf Compare June 26, 2024 10:06
@mmarchois mmarchois merged commit d6088a8 into master Jun 28, 2024
1 of 2 checks passed
@mmarchois mmarchois deleted the feat-mm-notifications branch June 28, 2024 09:18
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