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 added this to Reviewing in Consul Democracy via automation Jul 8, 2021
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
Consul Democracy automation moved this from Reviewing to Testing 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
Consul Democracy automation moved this from Testing to Release 1.4.0 Jul 9, 2021
@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.

None yet

2 participants