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

support form: integrate zenodo implementation #71

Closed
3 tasks done
kpsherva opened this issue Oct 27, 2022 · 0 comments · Fixed by #61
Closed
3 tasks done

support form: integrate zenodo implementation #71

kpsherva opened this issue Oct 27, 2022 · 0 comments · Fixed by #61

Comments

@kpsherva
Copy link
Contributor

kpsherva commented Oct 27, 2022

Information

See https://github.com/zenodo/rdm-project/issues/7

Actions

  • FIX draft PR: Add custom code for support #61
  • Evaluate what needs to be done to move the support form to InvenioRDM e.g migrate bootstrap to SUI, angular to react etc.
  • Evaluate if it makes sense to move the support template to InvenioRDM and have each instance provide its own template or only parts of it needs to be configurable

The HTML template of the form is defined in: zenodo/modules/support/templates/zenodo_support/contact_form.html

  1. The sanitization of the form data is carried on in a script defined in the same file.

The function dedicated to handling the submit and rendering the template is on: zenodo/modules/support/views.py

  1. The categories of the support form are defined in the configuration of the application.
  2. The handling of the template consists of performing a last check of the data, then wrap it and send it in two emails, one confirmation email for the user and another support ticket for Zenodo.
@kpsherva kpsherva linked a pull request Oct 27, 2022 that will close this issue
@alejandromumo alejandromumo removed their assignment Nov 11, 2022
@jrcastro2 jrcastro2 assigned jrcastro2 and unassigned jrcastro2 Nov 14, 2022
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 a pull request may close this issue.

3 participants