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

Personal account #1600

Merged
merged 1 commit into from
Feb 28, 2017
Merged

Conversation

sleshchenko
Copy link
Contributor

@sleshchenko sleshchenko commented Jan 25, 2017

What does this PR do?

Makes personal account as onpremises packaging specific feature.
Reworks organization manager to publish cascade events in manager's method in transaction.
Move logic related to cascade removing of members and suborganization from event listeners to organization manager
Add missed indexes for foreign keys to account id.

What issues does this PR fix or reference?

#1025

Changelog

  • Add personal account for team capabilitities

Release Notes

  • N/A (part of teams)

Docs PR

N/A (part of teams)

Tests written?

Yes

import static com.codenvy.organization.api.permissions.OrganizationDomain.MANAGE_RESOURCES;
import static com.codenvy.organization.api.permissions.OrganizationDomain.MANAGE_WORKSPACES;

@Singleton
Copy link
Contributor

Choose a reason for hiding this comment

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

javadoc

permissionsFilter.checkNamespaceAccess(subject, "namespace");
}

// @Test
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Uncommented and fixed

@sleshchenko sleshchenko changed the title [WIP] CODENVY-1025 Personal organization CODENVY-1025 Personal organization Jan 27, 2017
@sleshchenko sleshchenko force-pushed the personalOrganization branch 2 times, most recently from 4aebaa1 to b1046f3 Compare February 2, 2017 08:39
@sleshchenko sleshchenko force-pushed the personalOrganization branch 6 times, most recently from de25962 to 5db866e Compare February 17, 2017 13:55
@sleshchenko sleshchenko force-pushed the personalOrganization branch 4 times, most recently from 6e5bd94 to 78fb38b Compare February 22, 2017 10:06
@sleshchenko sleshchenko force-pushed the personalOrganization branch 4 times, most recently from 5584571 to 14c5320 Compare February 28, 2017 13:04
@slemeur slemeur changed the title CODENVY-1025 Personal organization Personal organization Feb 28, 2017
@ashumilova ashumilova added this to the 5.4.0 milestone Feb 28, 2017
@sleshchenko sleshchenko self-assigned this Feb 28, 2017
@slemeur slemeur changed the title Personal organization Personal account Feb 28, 2017
@sleshchenko sleshchenko merged commit 51af057 into codenvy:master Feb 28, 2017
@sleshchenko sleshchenko deleted the personalOrganization branch February 28, 2017 18:10
@JamesDrummond JamesDrummond mentioned this pull request Mar 8, 2017
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants