-
Notifications
You must be signed in to change notification settings - Fork 50
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
Implement api_download_file() in views/findings.py #76
Comments
Hi, I am new to open source, and I would like to take up this issue. Could please brief me about what exactly the issue is? |
Hello @Pranav25191 In the The functionality will be slightly similar to the method below, Ideally, i'd recommend CSV or JSON as the output of this specific API call. @scovetta thoughts on output? We welcome contributions to the repo. Note that the changes are still under review in a Pull request (PR) #55 so the source code will not be available in main until the PR has been approved and merged. Familiarity with python and django framework will be useful for this issue. To start: Let me know if you have any questions. I will update the ticket with further details on acceptance criteria and expected outputs. |
I think this part is pretty core to the "how should we store/access files for the UI?" -- it's a lot more complicated than it seemed early on (surprise!). I'm going through comments on #55 today and tomorrow and hoping to my thoughts in order to give better guidance here -- so if you don't mind, come back to this thread in a few days and I'll have better suggestions. |
@scovetta @Cyber-JiuJiteria I've learnt Django Recently and would like to try this issue out if it's still open? I wonder if the project is active? Was looking to apply for the mentorship program mentioned here -: https://mentorship.lfx.linuxfoundation.org/project/4df8fab8-e11a-4877-8140-c3633099ea24 |
hello @ShivamMenda thanks for reaching out with interest. The project is still active. We are a small team and have several repos we are working on. Below are some good examples on how to help. Check out the PR #123 for the frontend for the Assertion Framework. Bring down this branch, build, run, and test it on your local. This PR 123 is still under review and we are looking for any feedback or enhancements for it. If you'd like to work on an issue, check the issues page for any issues with 'good first issue,' and 'help wanted.' labels, such as #92 . We are available on the OpenSSF Slack via #alpha-omega if you have any questions. |
@Cyber-JiuJiteria Thank you for replying I'll join the slack Channel and talk to you guys more there. I'll checkout the codebase and get familiar with it. I'll also apply for the mentorship program and start contributing. |
@Cyber-JiuJiteria can you add me on the slack channel? My email is shivammenda2002@gmail.com |
@ShivamMenda Try going through the Slack link here https://www.openssf.org/getinvolved I do not have a way to share outside of the link on the website. |
@Cyber-JiuJiteria Sure I'll try it out and let you know! |
@Cyber-JiuJiteria I've joined the slack workspace and will be communicating over there too. Thank you. |
|
Need to define the output file types that will be provided (json, csv, pdf)
Originally posted by @Cyber-JiuJiteria in #55 (comment)
The text was updated successfully, but these errors were encountered: