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.

Sign up for GitHub

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

Jump to bottom

[IMPAC-335] Multiple Recipients #125

Open
wants to merge 1 commit into
base: 1.0
Choose a base branch
from
Open

[IMPAC-335] Multiple Recipients #125

wants to merge 1 commit into from

Conversation

adamaziz15
Copy link
Contributor

No description provided.

@xaun xaun mentioned this pull request Jan 18, 2017
Merged
@adamaziz15 adamaziz15 changed the title [IMPAC-335] Multiple Recipients [IMPAC-335] [WIP] Multiple Recipients Jan 18, 2017
@adamaziz15 adamaziz15 force-pushed the feature/335-multiple-recipients branch from ecfd950 to 640c211 Compare January 24, 2017 16:23
@adamaziz15 adamaziz15 changed the base branch from master to 1.0 January 24, 2017 16:24
MichaelSedov pushed a commit to MichaelSedov/mno-enterprise-angular that referenced this pull request Jan 26, 2017
@ouranos
Merge pull request maestrano#125 from clemthenem/improvment/purchase_…
e7f74b1 
…workflow

[MNOE-170] - Purchase Workflow
@adamaziz15 adamaziz15 changed the title [IMPAC-335] [WIP] Multiple Recipients [IMPAC-335] Multiple Recipients Jan 26, 2017
@adamaziz15
Copy link
Contributor Author

@alexnoox @cesar-tonnoir @xaun Please Review. This goes along with PR's for [IMPAC-335] in impac-angular, mnoe, and mno-hub.

alexnoox
alexnoox approved these changes Jan 30, 2017
View reviewed changes
Copy link
Contributor

@alexnoox alexnoox left a comment

Choose a reason for hiding this comment

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

LGTM

@cesar-tonnoir Can you review this as well?

@cesar-tonnoir cesar-tonnoir self-requested a review January 30, 2017 04:32
cesar-tonnoir
cesar-tonnoir approved these changes Jan 30, 2017
View reviewed changes
Copy link
Contributor

@cesar-tonnoir cesar-tonnoir left a comment

Choose a reason for hiding this comment

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

👍

@adamaziz15 adamaziz15 force-pushed the feature/335-multiple-recipients branch from 640c211 to 1f9eeb9 Compare January 30, 2017 19:33
@cesar-tonnoir
Copy link
Contributor

cesar-tonnoir commented Feb 20, 2017 •
edited
Loading

@alexnoox can you please merge to the equivalent of your develop branch, and let us know when we can deploy and test in uat ?

@adamaziz15 adamaziz15 force-pushed the feature/335-multiple-recipients branch from 1f9eeb9 to ce6d4fc Compare February 20, 2017 16:05
@xaun
Copy link
Contributor

xaun commented Feb 21, 2017

NOTE: these changes are dependant on: maestrano/mno-enterprise#208
Which means it cannot be merged until MNOE is merged and tagged, and mno-hub is deployed to prod on all instances. @alexnoox @ouranos @cesar-tonnoir

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cesar-tonnoir cesar-tonnoir cesar-tonnoir approved these changes

@alexnoox alexnoox alexnoox approved these changes

Assignees

@alexnoox alexnoox

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@adamaziz15 @cesar-tonnoir @xaun @alexnoox