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: Add SeenBy per user #368

Merged
merged 15 commits into from
Feb 1, 2024
Merged

feat: Add SeenBy per user #368

merged 15 commits into from
Feb 1, 2024

Conversation

oskogstad
Copy link
Collaborator

@oskogstad oskogstad commented Jan 21, 2024

Adds SeenBy per user in activities

@oskogstad oskogstad force-pushed the feature/seen-by-per-user branch 4 times, most recently from f26ebcd to 03d87a5 Compare January 31, 2024 15:46
@oskogstad oskogstad force-pushed the feature/seen-by-per-user branch from 03d87a5 to 06cfce6 Compare January 31, 2024 16:00
@oskogstad oskogstad marked this pull request as ready for review January 31, 2024 16:44
@oskogstad
Copy link
Collaborator Author

oskogstad commented Jan 31, 2024

@MagnusSandgren kommenterte på at IUserService har for mye ansvar (blir utvidet i denne PRen), laget et issue på det:
#403

elsand
elsand previously approved these changes Feb 1, 2024
Copy link
Member

@elsand elsand left a comment

Choose a reason for hiding this comment

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

lgtm

@elsand elsand dismissed their stale review February 1, 2024 07:50

Misclick, må få fjernet description-innslag

Copy link

sonarqubecloud bot commented Feb 1, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

8 New issues
0 Security Hotspots
No data about Coverage
0.6% Duplication on New Code

See analysis details on SonarCloud

@oskogstad oskogstad merged commit c68db9e into main Feb 1, 2024
5 checks passed
@oskogstad oskogstad deleted the feature/seen-by-per-user branch February 1, 2024 15:36
oskogstad added a commit that referenced this pull request Feb 2, 2024
arealmaas pushed a commit that referenced this pull request Feb 28, 2024
🤖 I have created a release *beep* *boop*
---


## 1.0.0 (2024-02-28)


### Features

* Add element count to eu list dto
([#414](#414))
([934fa93](934fa93))
* Add purge functionallity separate from soft delete.
([#483](#483))
([1349efb](1349efb))
* Add SeenBy per user
([#368](#368))
([c68db9e](c68db9e))
* **azure:** parameterize SKUs
([#364](#364))
([9c27c74](9c27c74))
* change format of party identifier
([#376](#376))
([27e6744](27e6744)),
closes [#220](#220)
* Container app revision verification on deploy
([#392](#392))
([db13a89](db13a89))
* Slack notifier IaC
([#341](#341))
([80c3579](80c3579))


### Bug Fixes

* 412 status on multiple requests without revision header
([#427](#427))
([047cf71](047cf71))
* add APIM base uri for dialogporten
([948b9a4](948b9a4))
* add apim base uri for staging
([#451](#451))
([580d946](580d946))
* add base uri for web api
([#425](#425))
([0aa941b](0aa941b))
* add correct APIM base uri for dialogporten
([713771a](713771a))
* add correct baseuri for altinn events
([#496](#496))
([74940ab](74940ab))
* Add null checks, set lists to empty if null
([#434](#434))
([f264aec](f264aec))
* **azure:** fix postgresql auth config
([#357](#357))
([4a4757f](4a4757f))
* **azure:** remove default value for KEY_VAULT_SOURCE_KEYS
([#418](#418))
([b0d74e8](b0d74e8))
* **azure:** remove default values in params and ensure secure on params
([#415](#415))
([94b9885](94b9885))
* **azure:** rename and fix outputs and pass correct secrets
([#416](#416))
([68f0c8b](68f0c8b))
* build errors for 8.0.200
([#440](#440))
([b133f8f](b133f8f))
* Check Content for null, use DependentRules, disallow empty
localization values
([#413](#413))
([894644a](894644a))
* Correct params for revision verification
([#405](#405))
([4b98348](4b98348))
* Do not allow empty content
([#436](#436))
([a083544](a083544))
* do not prefix swagger document in development
([#491](#491))
([e330ce3](e330ce3))
* remove path to swagger json
([fe1e770](fe1e770))
* rename migration job
([#423](#423))
([3897db2](3897db2))
* restrict container apps to apim ip
([#448](#448))
([1a1f3ad](1a1f3ad))
* Return 410 Gone when updating deleted dialog
([#464](#464))
([2498b0a](2498b0a))
* set base path for swagger json ui
([476fdca](476fdca))
* set base url for swagger json
([#447](#447))
([2161066](2161066))
* shorten secret name for container app job
([#422](#422))
([09b2f30](09b2f30))
* try echoing pgpassword in migration job🤫
([#419](#419))
([fe673a3](fe673a3))
* Use data from events, not from db
([#455](#455))
([469c606](469c606))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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