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

Constraints management in hermes-console #1106

Merged
merged 17 commits into from
Oct 1, 2019

Conversation

jewertow
Copy link
Contributor

@jewertow jewertow commented Sep 23, 2019

Hermes console allows to create, edit and delete constraints for topics and subscriptions.

Due to the lack of support for roles in hermes-console home view has no redirection button to constraints list. In order to go to constraints list it's necessary to manually redirect to #/constraints.

Screenshot 2019-09-23 at 11 20 27

@krzysztofcybulski
Copy link
Contributor

krzysztofcybulski commented Sep 23, 2019

Shouldn't constraints be visible in topic/subscription views rather than in the new one?

@jewertow
Copy link
Contributor Author

@krzysztofpcy we assumed to display topics and subscriptions separately.

* Add button to remove constraints.

* Redirect to /constraints-list after successful remove operation.

* Add ability to edit existing constraints.

* Add ability to create new constraints.

* Fix add and edit operations.

* Fix remove operation.

* Handle dismissed remove request.

* Improve info in modal.

* UX improvements

* UX improvements

* Add forms validation.
@jewertow jewertow changed the title Display constraints in hermes-console Constraints management in hermes-console Sep 30, 2019
</h4>
</div>
<a class="list-group-item" ng-repeat="item in topicConstraints" ng-click="edit(item)">
<small>{{$index + 1}}</small>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

@dankraw dankraw merged commit ec31b2e into master Oct 1, 2019
@cristaloleg cristaloleg deleted the hermes-console-constraints-management branch November 27, 2019 12:40
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.

None yet

4 participants