Refactor #17844
unify js code for email test buttons
| Status: | Closed | ||
|---|---|---|---|
| Priority: | Normal | ||
| Assignee: |  Tomer Brisker | ||
| Category: | |||
| Target version: | 1.15.0 | ||
| Difficulty: | Team Backlog: | ||
| Triaged: | Fixed in Releases: | ||
| Bugzilla link: | Found in Releases: | ||
| Pull request: | https://github.com/theforeman/foreman/pull/4159 | ||
Description
There is duplicate code in users.js and settings.js for the mail test button.
They should be unified, also possibly making the button a react component.
Associated revisions
Fixes #17844 - unify test mail js code
History
#1
Updated by The Foreman Bot over 1 year ago
- Status changed from New to Ready For Testing
- Assignee set to Tomer Brisker
- Pull request https://github.com/theforeman/foreman/pull/4159 added
#2 Updated by Anonymous over 1 year ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset de3759ffaf7ae7cd4d709f3a9f7e433194ef7305.
#3
Updated by Dominic Cleal over 1 year ago
- Legacy Backlogs Release (now unused) set to 209
#4
Updated by Daniel Lobato Garcia over 1 year ago
- Target version set to 169
#5
Updated by Brad Buckingham over 1 year ago
- Target version deleted (
169)
#6
Updated by Daniel Lobato Garcia over 1 year ago
- Target version set to 1.11.0