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

add Account Manager Page #93

Merged
merged 89 commits into from
Aug 10, 2023
Merged

Conversation

sanglevinh
Copy link
Collaborator

No description provided.

@sanglevinh sanglevinh linked an issue Aug 2, 2023 that may be closed by this pull request
@sanglevinh sanglevinh force-pushed the feature/account_manager branch from a3f4c32 to 769cdd4 Compare August 2, 2023 10:23
@sanglevinh sanglevinh force-pushed the feature/account_manager branch from b1e966d to 6500212 Compare August 2, 2023 11:28
# Conflicts:
#	frontend/src/@types/index.ts
#	frontend/src/api/users/UsersApiDTO.ts
#	frontend/src/pages/Dashboard/index.tsx
#	frontend/src/pages/Workspace/index.tsx
#	frontend/src/store/slice/User/UserActions.ts
#	frontend/src/store/slice/User/UserSlice.ts
#	frontend/src/store/slice/User/UserType.ts
@sanglevinh sanglevinh force-pushed the feature/account_manager branch from 1d26011 to 16205e0 Compare August 3, 2023 07:25
…dd_account

# Conflicts:
#	frontend/src/pages/AccountManager/index.tsx
…nt_manager_edit_account

# Conflicts:
#	frontend/src/pages/AccountManager/index.tsx
@sanglevinh
Copy link
Collaborator Author

sorry, I’ve overlooked.

access to /console/account-manager directory, we can see the page. tables are empty, but non-admin users should be redirected to /console.
@ReiHashimoto
I fixed it.

Copy link

@ReiHashimoto ReiHashimoto left a comment

Choose a reason for hiding this comment

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

LGTM

@sanglevinh sanglevinh force-pushed the feature/account_manager branch from ca218ce to bcdccfe Compare August 8, 2023 03:40
@sanglevinh sanglevinh marked this pull request as ready for review August 9, 2023 04:44
@itutu-tienday itutu-tienday merged commit 0b0f09e into develop-main Aug 10, 2023
sanglevinh pushed a commit that referenced this pull request Oct 16, 2023
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.

6. Account Manager
4 participants