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

Remove ES6-specific syntax in amsify suggestags #4583

Merged
merged 1 commit into from
Jul 9, 2021
Merged

Conversation

javierm
Copy link
Member

@javierm javierm commented Jul 8, 2021

References

Objectives

  • Make sure JavaScript loads in old browsers which don't support the ES6 syntax.

Notes

We don't officially guarantee all JavaScript will work on old browsers, but we do our best to make the page usable to a reasonable degree on them because the main goal of CONSUL is to help organizing participatory processes where all citizens can participate.

We're getting warnings from ESLint. However, we're ignoring them since this is a vendor file and follows its own conventions.

@javierm javierm added the UX label Jul 8, 2021
@javierm javierm self-assigned this Jul 8, 2021
@javierm javierm force-pushed the remove_es6_syntax branch from 60d3eeb to abc659a Compare July 8, 2021 17:42
@javierm javierm force-pushed the remove_es6_syntax branch from abc659a to 7d97e4c Compare July 8, 2021 17:45
@javierm javierm force-pushed the remove_es6_syntax branch from 7d97e4c to 4153e52 Compare July 8, 2021 17:46
It was causing JavaScript to stop working completely on old browsers
which don't support this syntax.
@taitus taitus self-assigned this Jul 9, 2021
@javierm javierm merged commit 7324e34 into master Jul 9, 2021
@javierm javierm deleted the remove_es6_syntax branch July 9, 2021 14:40
@javierm javierm added the Bug label Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants