Skip to content

Just the forms plugin [Experiment, do not merge]#76

Open
markpatton wants to merge 3 commits intomasterfrom
justforms
Open

Just the forms plugin [Experiment, do not merge]#76
markpatton wants to merge 3 commits intomasterfrom
justforms

Conversation

@markpatton
Copy link
Collaborator

@markpatton markpatton commented Jul 20, 2021

An an experiment, clean out various hacks and try just running the forms plugin without appointment or any other complications. Unfortunately many of the same issues described in #75 are also present.

Issues observed:

  • Various English translations needed.
  • On manage plugins page, Uncaught ReferenceError: switchList is not defined
    at HTMLDocument. (ManagePlugins.jsp?plugin_name=:1074)
  • Tries to load the js library toastr from the wrong location. Attempts /js/admin/jquery/plugins/toastr/toastr.min.js but is actually js/jquery/plugins/toastr/toastr.min.js
  • Attempts to load http://localhost:8080/js/admin/locales/bootstrap-flatpickr..js because the macro to load flatpickr doesn’t handle <@getRegional language=language /> returning an empty string
  • Date picker does not work when a date question is added, probably related to above.
    • Interestingly date pickers to select a range when viewing form answers work and are localized
  • When clicking on a question to edit it, this exception shows up in the console, but the edit seems to work
    • Uncaught SyntaxError: Unexpected identifier ManageQuestions.jsp?view=modifyQuestion&id_step=5&id_question=17:790
  • When using a form, these files are not found:

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