Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

Feature/lattice 2228 external atlas access #359

Merged
merged 78 commits into from
Nov 27, 2019

Conversation

pckeyes
Copy link
Contributor

@pckeyes pckeyes commented Oct 8, 2019

Creation and handling of securable object types that represent tables and columns of an organization's atlas database. These new securable objects will allow for creation of acl keys to enable granting permissions at the table or column level within atlas.

Also apis for granting permissions, updating table/column names

Piper Keyes added 3 commits November 5, 2019 14:00
…om:openlattice/api into feature/LATTICE-2228-external-atlas-access

merge
companion object {
/* These determine the service routing */
const val SERVICE = "/datastore"
const val CONTROLLER = "/organization-database"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is just a general comment, but we've been super inconsistent in these controller paths between different patterns of first-second vs first_second vs firstsecondvs first/second so we should probably collectively pick whichever one we want and stick to that pattern moving forward

Copy link
Contributor

@katherinebernstein katherinebernstein left a comment

Choose a reason for hiding this comment

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

just a few minor things!

Copy link
Contributor

@geekbeast geekbeast left a comment

Choose a reason for hiding this comment

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

Minor change with companion object.

src/main/kotlin/com/openlattice/organization/DatasetApi.kt Outdated Show resolved Hide resolved
@geekbeast geekbeast merged commit 8bf7d6b into develop Nov 27, 2019
@geekbeast geekbeast deleted the feature/LATTICE-2228-external-atlas-access branch November 27, 2019 02:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants