Skip to content
This repository was archived by the owner on Apr 19, 2023. It is now read-only.

feat(mdc) : Add MDC Application Dash#836

Closed
camilamacedo86 wants to merge 1 commit intointegr8ly:add-mobile-servicesfrom
camilamacedo86:AEROGEAR-9650
Closed

feat(mdc) : Add MDC Application Dash#836
camilamacedo86 wants to merge 1 commit intointegr8ly:add-mobile-servicesfrom
camilamacedo86:AEROGEAR-9650

Conversation

@camilamacedo86
Copy link
Copy Markdown
Contributor

Additional Information

https://issues.jboss.org/browse/AEROGEAR-9650

What

Re-applying changes since it was missed in some rebase with the master in this branch.

@wei-lee
Copy link
Copy Markdown
Contributor

wei-lee commented Aug 12, 2019

@camilamacedo86 I am not sure why it is show up but this PR is already landed in the feature branch. See https://github.com/integr8ly/installation/blob/add-mobile-services/roles/mdc/defaults/main.yml

@wei-lee
Copy link
Copy Markdown
Contributor

wei-lee commented Aug 12, 2019

This PR is in the feature branch anyway, so I am going to close this.

@wei-lee wei-lee closed this Aug 12, 2019
@camilamacedo86
Copy link
Copy Markdown
Contributor Author

camilamacedo86 commented Aug 12, 2019

Hi @wei-lee but if it was fully there, so why it shows the differences between both?
Could you check the files changed option?

@grdryn
Copy link
Copy Markdown
Member

grdryn commented Aug 13, 2019

@camilamacedo86 I believe that might be because your branch is stale (it's based on an old master, rather than the add-mobile-services branch). If you check out your branch locally and try to rebase it on top of the origin/add-mobile-services branch, you'll notice that you just get fast-forwarded to the tip of the origin/add-mobile-services branch, and no new commit is added on top, because the changes already exist on that branch.

I guess GitHub just doesn't display the diff very well when the commits are this far apart or something. I'm not sure though.

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

Labels

mobile RHMDS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants