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

Added SRE meeting types to db and graph #264

Merged
merged 3 commits into from
Jan 24, 2024
Merged

Added SRE meeting types to db and graph #264

merged 3 commits into from
Jan 24, 2024

Conversation

Muncherkin
Copy link
Contributor

@Muncherkin Muncherkin commented Aug 31, 2023

Added database and graph support for SRE meetings of type "grundnivå", "avancerad nivå" and "intern".

Other than the main feature:

  • Added myself to the contributor list
  • Removed two broken tests from user.api.ts which I have spoken to Froborg about
  • Fixed an error message that was missing the word "har"
  • Redid the name assignment logic in the meeting reducer
  • Fixed the prisma:reset commad for Windows users (without destroying it for linux chads)

Note this change is dependant on PR #136 in efterphest.

@github-actions
Copy link

github-actions bot commented Aug 31, 2023

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 67.13% 1350/2011
🔴 Branches 44.26% 216/488
🔴 Functions 57.38% 241/420
🟡 Lines 66.88% 1280/1914

Test suite run success

236 tests passing in 19 suites.

Report generated by 🧪jest coverage report action from 58e334e

@github-actions
Copy link

I have deployed this PR to feat-sreminutes-ekorre.review.esek.se 🚀

package.json Outdated Show resolved Hide resolved
src/reducers/meeting.reducer.ts Outdated Show resolved Hide resolved
@Muncherkin Muncherkin merged commit af66ce8 into main Jan 24, 2024
7 checks passed
Copy link

I have removed this deploy now 😇

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