Feature #12518

Add a toggle for previous partners

Added by Anonymous 2017-05-05 14:30:45 . Updated 2017-09-18 12:25:33 .

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Fundraising
Target version:
Tails_3.2
Start date:
2017-05-05
Due date:
% Done:

100%

Feature Branch:
451f:tails/feature/12518+toggle_partners
Type of work:
Website
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

.. to the partners page.
For now, let’s leave it as is.

Subtasks

Related issues

Related to Tails - Feature #6972: Create a "Partners" page Resolved 2017-05-05
Blocks Tails - Bug #12646: Make the Partners pages translatable Resolved 2017-06-06

History

#1 Updated by emmapeel 2017-05-25 08:08:13

  • Status changed from New to Confirmed

#2 Updated by intrigeri 2017-06-02 16:19:51

  • Tracker changed from Bug to Feature
  • Category set to Fundraising

#3 Updated by Anonymous 2017-06-06 17:08:12

#4 Updated by Anonymous 2017-06-06 17:08:22

#5 Updated by Anonymous 2017-06-14 12:59:57

  • Status changed from Confirmed to In Progress
  • Assignee set to sajolida
  • % Done changed from 0 to 30
  • QA Check set to Ready for QA
  • Feature Branch set to 451f:tails/feature/12518+toggle_partners

Please review.

#6 Updated by Anonymous 2017-06-14 13:03:53

  • blocks Bug #12646: Make the Partners pages translatable added

#7 Updated by Anonymous 2017-07-10 10:31:25

  • Assignee changed from sajolida to intrigeri

I’m reassigning this for review to intrigeri so that we can move forward on Bug #12646.

#8 Updated by Anonymous 2017-07-10 10:32:37

  • Target version set to Tails_3.2

#9 Updated by intrigeri 2017-07-13 09:41:09

  • Assignee deleted (intrigeri)
  • QA Check changed from Ready for QA to Info Needed

Why are we using new, custom JS instead of https://ikiwiki.info/ikiwiki/directive/toggle/ as we do everywhere else on our website? I see no justification for this in the commit log, hence my question :)

#10 Updated by Anonymous 2017-09-05 19:41:22

I was not aware this existed.

#11 Updated by Anonymous 2017-09-12 16:44:28

Looks like this does not work.. I still need to find out why but the directive is simply not translated into usable HTML.

#12 Updated by Anonymous 2017-09-18 09:43:24

  • Assignee set to intrigeri
  • QA Check changed from Info Needed to Ready for QA

Seems like the problem was simply due to a faulty syntax.

#13 Updated by intrigeri 2017-09-18 12:24:43

  • Status changed from In Progress to Resolved
  • % Done changed from 30 to 100

Applied in changeset commit:99fa75bb231a0b14ac39083e2081bdbdbf55dd15.

#14 Updated by intrigeri 2017-09-18 12:25:33

  • Assignee deleted (intrigeri)
  • QA Check changed from Ready for QA to Pass

looks nice!