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.

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

Guidance For Contributing JSON Parsing #201

Open
scottcc opened this issue Sep 2, 2020 · 4 comments
Open

Guidance For Contributing JSON Parsing #201

scottcc opened this issue Sep 2, 2020 · 4 comments
Assignees
Milestone

Comments

@scottcc
Copy link

scottcc commented Sep 2, 2020

Hey there. Just looking for some pointers on how one would help enhance this plugin to render "prettified" JSON via this excellent plugin.

The reasoning is to work nicely with this commercial Redmine Plugin: Checklists: https://www.redmineup.com/pages/plugins/checklists

This is what it currently "shows up" like:

gitlab-mattermost-redmineup

So - looking for quick recommendations on where in the code would be the best place to start looking, and if there are particular JSON parsing libraries that are recommended (or already in use!), we would highly appreciate knowing the info.

@tw-ayush
Copy link

tw-ayush commented Oct 6, 2020

the following should solve the problem for you.

    error := json.Indent(&prettyJSON, body, "", "\t")
    if error != nil {
        log.Errorf("JSON parse error: ", error)
        App.BadRequest(w)
        return
    }

@scottcc
Copy link
Author

scottcc commented Oct 16, 2020

Thank you! Now... where to integrate this in...

@mickmister
Copy link
Contributor

@DHaussermann to invesigate applicable scenario

@mickmister mickmister added this to the Planned Work milestone Jan 26, 2022
@masy8
Copy link

masy8 commented Apr 24, 2023

How to get JSON data about checklist? @scottcc

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

No branches or pull requests

5 participants