Skip to content
This repository has been archived by the owner on Mar 27, 2023. It is now read-only.

Release #1188

Merged
merged 5 commits into from
Jun 14, 2018
Merged

Release #1188

merged 5 commits into from
Jun 14, 2018

Conversation

vincemtnz
Copy link
Contributor

Various bugfixes

  • Fix spacing in email bodies
  • Deprecates old fundraiser template (backbone)
  • Moves intl-locale globals to separate file, initialises them in member_facing.js
  • Fixes spacing between paragraphs in page body. Use enforce-styles to override.

rodrei and others added 5 commits June 14, 2018 11:21
Since removing simple_format, the text is no longer wrapped in <p></p>
We're seeing a lot of errors telling us that `$` is not a function. We suspect that something's messing up with jQuery but we're not entirely sure. This is a small hack to hopefully re-assign jQuery to the `$` variable. 🤞
* import I18n from 'champaign-i18n' (alias to window.I18n in webpack)

* Deprecate legacy backbone fundraiser code.

The _fundraiser_refactor.liquid partial has been deleted and its
contents are now in _fundraiser.liquid.

All templates that referenced 'Fundraiser Refactor' must be updated
to include 'Fundraiser'.

* Add global locale data in member_facing.js

Instead of doing this in ComponentWrapper, add it immediately in member_facing.js

* Removed unused lines

* Don't attempt to assign petitionForm to jXHR response if it's a string
@vincemtnz vincemtnz merged commit 5f33843 into master Jun 14, 2018
vincemtnz added a commit that referenced this pull request Jun 14, 2018
This reverts commit 5f33843.
vincemtnz added a commit that referenced this pull request Jun 14, 2018
This reverts commit 40ba347.
vincemtnz added a commit that referenced this pull request Jun 14, 2018
This reverts commit 40ba347.
vincemtnz pushed a commit that referenced this pull request Jun 18, 2018
* Don't run simple_format on emails body

* Update email spec

Since removing simple_format, the text is no longer wrapped in <p></p>

* [hack] check to see if $ is being clobbered somewhere. (#1185)

We're seeing a lot of errors telling us that `$` is not a function. We suspect that something's messing up with jQuery but we're not entirely sure. This is a small hack to hopefully re-assign jQuery to the `$` variable. 🤞

* Update pension emails spec

* Various bugfixes (#1187)

* import I18n from 'champaign-i18n' (alias to window.I18n in webpack)

* Deprecate legacy backbone fundraiser code.

The _fundraiser_refactor.liquid partial has been deleted and its
contents are now in _fundraiser.liquid.

All templates that referenced 'Fundraiser Refactor' must be updated
to include 'Fundraiser'.

* Add global locale data in member_facing.js

Instead of doing this in ComponentWrapper, add it immediately in member_facing.js

* Removed unused lines

* Don't attempt to assign petitionForm to jXHR response if it's a string

* Redo: "Release (#1188)"

This reverts commit 40ba347.

* Set user context in sentry
vincemtnz pushed a commit that referenced this pull request Jul 3, 2018
* Don't run simple_format on emails body

* Update email spec

Since removing simple_format, the text is no longer wrapped in <p></p>

* [hack] check to see if $ is being clobbered somewhere. (#1185)

We're seeing a lot of errors telling us that `$` is not a function. We suspect that something's messing up with jQuery but we're not entirely sure. This is a small hack to hopefully re-assign jQuery to the `$` variable. 🤞

* Update pension emails spec

* Various bugfixes (#1187)

* import I18n from 'champaign-i18n' (alias to window.I18n in webpack)

* Deprecate legacy backbone fundraiser code.

The _fundraiser_refactor.liquid partial has been deleted and its
contents are now in _fundraiser.liquid.

All templates that referenced 'Fundraiser Refactor' must be updated
to include 'Fundraiser'.

* Add global locale data in member_facing.js

Instead of doing this in ComponentWrapper, add it immediately in member_facing.js

* Removed unused lines

* Don't attempt to assign petitionForm to jXHR response if it's a string

* Redo: "Release (#1188)"

This reverts commit 40ba347.

* Set user context in sentry
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants