Skip to content
This repository has been archived by the owner on Dec 23, 2019. It is now read-only.

Bugfix/delete menu action #801

Merged
merged 2 commits into from
Jul 26, 2018
Merged

Bugfix/delete menu action #801

merged 2 commits into from
Jul 26, 2018

Conversation

radireddy
Copy link
Contributor

Requirements

  • Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
  • All new code requires tests to ensure against regressions

Title of the Pull Requests

Delete interactive menu does not work

Description of the Change

On clicking delete interactive menu has no effect. No success or error message is shown. API call was not made on clicking the confirm button. I have made the required code change to make the API call.

Benefits

fixes bug MFRWW-609

Possible Drawbacks

none

Applicable Issues

MFRWW-609

Ravindra Adireddy added 2 commits July 26, 2018 10:08
clicking on delete interative menu link has no effect. It does not show success or error message.

CLoses #MFRWW-609
clicking on delete interative menu link has no effect. It does not show success or error message.

CLoses #MFRWW-609
@antleblanc
Copy link
Contributor

@radireddy your branch isn't rebased on master that's why there are 14 commits. Could you please rebase on master?

@antleblanc antleblanc added the bug label Jul 26, 2018
@radireddy radireddy changed the base branch from master to develop July 26, 2018 07:20
@antleblanc antleblanc merged commit bbcb7ef into develop Jul 26, 2018
@antleblanc antleblanc deleted the bugfix/delete-menu-action branch July 26, 2018 09:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

3 participants