Skip to content

[VET-1380] Add admin/status/whoami#271

Merged
anneeb merged 4 commits intomasterfrom
feature/VET-1380-add-admin-status-whoami
Jun 3, 2022
Merged

[VET-1380] Add admin/status/whoami#271
anneeb merged 4 commits intomasterfrom
feature/VET-1380-add-admin-status-whoami

Conversation

@SpiralP
Copy link
Copy Markdown
Member

@SpiralP SpiralP commented Jun 1, 2022

Copy link
Copy Markdown
Contributor

@anneeb anneeb 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
Copy Markdown
Contributor

@hasibhassan hasibhassan left a comment

Choose a reason for hiding this comment

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

LGTM!

@SpiralP SpiralP marked this pull request as draft June 2, 2022 17:04
@SpiralP
Copy link
Copy Markdown
Member Author

SpiralP commented Jun 2, 2022

going to add tests!

@SpiralP SpiralP marked this pull request as ready for review June 3, 2022 04:46
@anneeb anneeb merged commit 0bb8adb into master Jun 3, 2022
@anneeb anneeb deleted the feature/VET-1380-add-admin-status-whoami branch June 3, 2022 17:58
anneeb added a commit that referenced this pull request Jun 3, 2022
* [VET-1380] Add `admin/status/whoami` to Stardog.js

* camel case

* add test for `user.whoAmI`

Co-authored-by: Annee Barrett <annee.barrett@stardog.com>
@anneeb anneeb modified the milestones: 4.1.0, 3.5.1 Jun 3, 2022
@anneeb anneeb changed the title [VET-1380] Add admin/status/whoami to Stardog.js [VET-1380] Add admin/status/whoami Jun 3, 2022
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.

4 participants