Skip to content

#403 - Added code to support configuring application name for XML Fil… #409

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
Open
wants to merge 2 commits into
base: 1x
Choose a base branch
from
Open

#403 - Added code to support configuring application name for XML Fil… #409

wants to merge 2 commits into from

Conversation

@vyasabhishek
Copy link

@vyasabhishek vyasabhishek commented Nov 18, 2016

…e Error Log.

@ThomasArdal
Copy link
Contributor

ThomasArdal commented Mar 3, 2017

@atifaziz I know we talked about the challenges of having application name in XML files. But isn't this worth merging?

@vyasabhishek
Copy link
Author

vyasabhishek commented Mar 3, 2018

Hi @atifaziz @ThomasArdal . I would really love this change to be merged in. What's your take on it?

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

Reviewers

1 more reviewer

@Crowfoot Crowfoot Crowfoot approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vyasabhishek @ThomasArdal @Crowfoot @vyasab