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 EU endpoints for seen log #607

Merged
merged 3 commits into from
Apr 9, 2024
Merged

Conversation

oskogstad
Copy link
Collaborator

@oskogstad oskogstad commented Apr 7, 2024

  • DB Migration, changed our minds on the record name
  • EU endpoint for SeenLog (SO in next PR)
  • Rename string hasher class
  • Refactoring username/pid, fethcing both name and pid in one function

@oskogstad oskogstad force-pushed the feat/add-seen-log-endpoints branch from b88cf96 to 119b85b Compare April 9, 2024 12:33
@oskogstad oskogstad force-pushed the feat/add-seen-log-endpoints branch from 119b85b to 23f852f Compare April 9, 2024 12:42
@oskogstad oskogstad changed the title feat: Add endpoints for seen log feat: Add EU endpoints for seen log Apr 9, 2024
@oskogstad oskogstad marked this pull request as ready for review April 9, 2024 12:44
@oskogstad oskogstad requested a review from a team as a code owner April 9, 2024 12:44
arealmaas
arealmaas previously approved these changes Apr 9, 2024
Copy link
Collaborator

@arealmaas arealmaas left a comment

Choose a reason for hiding this comment

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

LGTM! 👏

Copy link

sonarcloud bot commented Apr 9, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
8.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@oskogstad oskogstad merged commit 1aa7eeb into main Apr 9, 2024
13 of 14 checks passed
@oskogstad oskogstad deleted the feat/add-seen-log-endpoints branch April 9, 2024 14:09
oskogstad added a commit that referenced this pull request Apr 9, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.4.0](v1.3.0...v1.4.0)
(2024-04-09)


### Features

* Split SeenLog from activities
([#598](#598))
([71b77d2](71b77d2))
* *This is a breaking change*, the `Seen` activity type has been
removed, and all activities of this type is removed from the staging
environment.
* Add EU endpoints for seen log
([#607](#607))
([1aa7eeb](1aa7eeb))
  * `/api/v1/enduser/dialogs/{dialogId}/seenlog`
  * `/api/v1/enduser/dialogs/{dialogId}/seenlog/{seenLogId}`
  * `/api/v1/serviceowner/dialogs/{dialogId}/seenlog`
  * `/api/v1/serviceowner/dialogs/{dialogId}/seenlog/{seenLogId}`
* Add ExtendedStatus content type
([#589](#589))
([a9f10b0](a9f10b0))
* add fusion cache
([#579](#579))
([973fa5c](973fa5c))
* **azure:** copy from keyvault to app config
([#593](#593))
([d216c90](d216c90))
* **service:** use in-memory transport instead of rabbitmq
([#602](#602))
([dc339e7](dc339e7))


### Bug Fixes

* Add PartyIdentifier.Separator to party validation error
([#595](#595))
([14ee4a1](14ee4a1))
* **azure:** ensure key vault url is correct and add keyvault readerrole
for migration job
([#597](#597))
([2f11a16](2f11a16))

---
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: Ole Jørgen Skogstad <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants