Skip to content

Conversation

@dregad
Copy link
Member

@dregad dregad commented Jun 10, 2017

  • display linked issues for each changeset
  • if no changeset is attached, add field to allow adding links without opening the view page

Fixes #202

@luzpaz testing and feedback would be appreciated.

@dregad
Copy link
Member Author

dregad commented Jun 26, 2017

Testing report by @kkremitzki

I threw that branch onto our staging environment and it looks good. I can attach, detach, use comma-separated lists of issues, so I'd say it hits all the "wants" for #202 and doesn't break under any of the ways I tried to use it

dregad added 3 commits June 26, 2017 16:30
If there are no issues linked to the changeset, display an input field
and button to allow the user to add links without having to open the
changeset view page.
Use changeset's revision id as anchor in the changesets list, and use
that as a new 'redirect' parameter for the attach page so it returns
straight to the changeset list's row that called it.

If the new param is not specified, the behavior is as it was before,
i.e. redirect to view page.
@dregad dregad merged commit 5aec110 into mantisbt-plugins:master Jun 26, 2017
@dregad dregad deleted the i202 branch June 26, 2017 14:33
@dregad
Copy link
Member Author

dregad commented Sep 14, 2017

This PR introduced a regression in View Issue page's Related Changesets section - see #231.

dregad added a commit that referenced this pull request Sep 14, 2017
When viewing an issue's Related Changesets (in view.php), an error
message 'String "plugin_SourceIntegration_affected_issues" not found.'
is displayed.

This is because the plugin context in View Issue page is not 'Source',
but 'SourceIntegration', so the plugin_lang_get() call needs to
reference the base plugin.

Regression introduced by 5aec110 (#224)

Fixes #231
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.

1 participant