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

Fix linting errors & use ES6 JS in EML 211 Editor View (Editor cleanup, part 1) #2566

Merged
merged 4 commits into from
Dec 17, 2024

Conversation

robyngit
Copy link
Member

@robyngit robyngit commented Nov 7, 2024

This PR is a simple code clean-up of the EML211EditorView. It doesn't change the logic or structure of the view, but just lints and modernizes the existing code. This is the first part of issue #2565

  • Fix all linting errors
  • Fix & add missing JSdocs
  • Remove unused render renderChildren function
  • Remove the editor.html template in favour of inline-template literals
  • Convert all underscore.js methods to modern ES6 methods

Remove unused render renderChildren function and fix JSdocs in EML211EditorView

Issue #2565
- Including replacing the underscore templates with ES6 template literals
- Remove the editor.html template file that is now unused.

Issue #2565, #2027
src/js/views/metadata/EML211EditorView.js Outdated Show resolved Hide resolved
src/js/views/metadata/EML211EditorView.js Outdated Show resolved Hide resolved
Missing from the template methods

Issue #2565
@robyngit robyngit merged commit 0445662 into develop Dec 17, 2024
1 check failed
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.

2 participants