Skip to content

Commit

Permalink
Merge pull request #26 from duckduckgo/jkt/setBreakageMaxLength
Browse files Browse the repository at this point in the history
Set breakage description maxlength 2500 to reduce chance of reports n…
  • Loading branch information
jonathanKingston authored Mar 30, 2023
2 parents 6824cda + 5049a5b commit 2ed4d2f
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 2 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,8 @@ jobs:
run: npm run prettier.check
- name: Run tests
run: npm run test.unit
- name: Clean tree
run: npm run test.clean-tree
build:
runs-on: ubuntu-20.04
needs: test
Expand Down
2 changes: 1 addition & 1 deletion build/app/public/js/base.js
Original file line number Diff line number Diff line change
Expand Up @@ -33055,7 +33055,7 @@ function _default() {
var icon = (0, _heroEs.largeHeroIcon)({
status: 'breakage-form'
});
return (0, _bel["default"])(_templateObject || (_templateObject = _taggedTemplateLiteral(["<section class=\"sliding-subview\">\n <div class=\"breakage-form\">\n ", "\n <div class=\"padding-x-double js-breakage-form-element\" data-state=\"idle\">\n <div class=\"key-insight\">\n ", "\n <div class=\"breakage-form__advise\">\n <p class=\"token-title-3\">", "</p>\n </div>\n <div class=\"breakage-form__message\">\n <p class=\"token-title-3-em\">", "</p>\n <p class=\"token-title-3\">", "</p>\n </div>\n </div>\n <div class=\"breakage-form__content\">\n <div class=\"breakage-form__element\">\n <div class=\"form__group\">\n <div class=\"form__select breakage-form__input--dropdown\">\n <select class=\"js-breakage-form-dropdown\">\n <option value=''>", "</option>\n ", "\n </select>\n </div>\n <textarea class=\"form__textarea js-breakage-form-description\" placeholder=\"", "\"></textarea>\n <button class=\"form__submit token-label-em js-breakage-form-submit\" role=\"button\">", "</button>\n </div>\n <div class=\"breakage-form__footer token-breakage-form-body\">\n ", "\n </div>\n </div>\n </div>\n </div>\n </div>\n </section>"])), (0, _topNav.topNav)({
return (0, _bel["default"])(_templateObject || (_templateObject = _taggedTemplateLiteral(["<section class=\"sliding-subview\">\n <div class=\"breakage-form\">\n ", "\n <div class=\"padding-x-double js-breakage-form-element\" data-state=\"idle\">\n <div class=\"key-insight\">\n ", "\n <div class=\"breakage-form__advise\">\n <p class=\"token-title-3\">", "</p>\n </div>\n <div class=\"breakage-form__message\">\n <p class=\"token-title-3-em\">", "</p>\n <p class=\"token-title-3\">", "</p>\n </div>\n </div>\n <div class=\"breakage-form__content\">\n <div class=\"breakage-form__element\">\n <div class=\"form__group\">\n <div class=\"form__select breakage-form__input--dropdown\">\n <select class=\"js-breakage-form-dropdown\">\n <option value=''>", "</option>\n ", "\n </select>\n </div>\n <textarea class=\"form__textarea js-breakage-form-description\" placeholder=\"", "\" maxlength=\"2500\"></textarea>\n <button class=\"form__submit token-label-em js-breakage-form-submit\" role=\"button\">", "</button>\n </div>\n <div class=\"breakage-form__footer token-breakage-form-body\">\n ", "\n </div>\n </div>\n </div>\n </div>\n </div>\n </section>"])), (0, _topNav.topNav)({
view: 'secondary'
}), icon, _localize.i18n.t('report:selectTheOptionDesc.title'), _localize.i18n.t('report:thankYou.title'), _localize.i18n.t('report:yourReportWillHelpDesc.title'), _localize.i18n.t('report:pickYourIssueFromTheList.title'), categories.map(function (item) {
return (0, _bel["default"])(_templateObject2 || (_templateObject2 = _taggedTemplateLiteral(["<option value=", ">", "</option>"])), item.value, item.category);
Expand Down
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@
"test.int.headed": "playwright test --headed",
"test.int.update-screenshots": "npm run test.int -- --update-snapshots",
"test.int-debug": "playwright test --headed --debug",
"test.clean-tree": "npm run build && sh scripts/check-for-changes.sh",
"verify.local": "npm run lint.fix; npm run prettier.fix; npm run test.unit; npm run build; npm run docs; npm run test.int"
},
"devDependencies": {
Expand Down
2 changes: 2 additions & 0 deletions scripts/check-for-changes.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
git update-index --refresh
git diff-index --quiet HEAD --
2 changes: 1 addition & 1 deletion shared/js/ui/templates/breakage-form.es6.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ export default function () {
</div>
<textarea class="form__textarea js-breakage-form-description" placeholder="${i18n.t(
'report:tellUsMoreDesc.title'
)}"></textarea>
)}" maxlength="2500"></textarea>
<button class="form__submit token-label-em js-breakage-form-submit" role="button">${i18n.t(
'report:sendReport.title'
)}</button>
Expand Down

0 comments on commit 2ed4d2f

Please sign in to comment.