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

added update restrictions, techStacks in project and orgId to a user #147

Merged
merged 3 commits into from
Jul 23, 2020

Conversation

Rupeshiya
Copy link
Member

@Rupeshiya Rupeshiya commented Jul 12, 2020


name: Pull Request template
about: Describe the PR.
title: ''
labels: ''
assignees: ''


Problem

Github Issue Number: #138
The issues are:

  1. Role-based invitation using the link
  2. Modify backend to store tech stacks for projects
  3. Admin mechanism in updates through org settings integration

_

Solution of problem

PR includes:

  • Backend for a role-based invite using the invite link
  • Modified projects section to store tech stacks used
  • Added admin mechanism in update restriction via org settings
  • Added error handler for max socket events

Type of Change

[✓] Bug fix
[ ] New Feature
[ ] Development of UI/UX prototypes
[✓] Small refactor
[ ] Change in Documentation

Checklist

[✓] My code follows the same style as the codebase
[ ] My Code change requires a change in documentation
[ ] I have updated the Readme accordingly
[✓] I made PR against development branch
[ ] I have run the test cases locally and it's passing.
[✓] I have squashed my commits

@Rupeshiya
Copy link
Member Author

@vaibhavdaren @devesh-verma Please review

app/controllers/organization.js Outdated Show resolved Hide resolved
app/controllers/user.js Outdated Show resolved Hide resolved
app/controllers/user.js Outdated Show resolved Hide resolved
@Rupeshiya Rupeshiya force-pushed the project_modification branch 3 times, most recently from a3d879d to eaf38a9 Compare July 20, 2020 10:16
@devesh-verma devesh-verma merged commit 92f7eef into codeuino:development Jul 23, 2020
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.

3 participants