Page MenuHomePhabricator

[website] Update the upgrade page
ClosedPublic

Authored by Fabien on Mar 7 2023, 13:24.

Details

Reviewers
PiRK
Mengerian
Group Reviewers
Restricted Project
Commits
rABCWEB4c158d76dc49: [website] Update the upgrade page
Summary

Update the upgrade page with the changes from the next release.

Test Plan
make serve

Check the upgrade page renders correctly and the content is correct.

Diff Detail

Repository
rABCWEB Bitcoin ABC Website
Branch
upgrade_page
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 22290
Build 44208: arc lint + arc unit

Event Timeline

Fabien requested review of this revision.Mar 7 2023, 13:24
PiRK added a subscriber: PiRK.
PiRK added inline comments.
upgrade.html
11 ↗(On Diff #38329)

nit: extra blank space

This revision is now accepted and ready to land.Mar 7 2023, 16:50

Need to update the upgrade notice also.

See D12036 for an example of what was done last time

upgrade.html
11 ↗(On Diff #38329)
Mengerian requested changes to this revision.Mar 7 2023, 17:47
This revision now requires changes to proceed.Mar 7 2023, 17:47

Also update the upgrade notice banner

Mengerian requested changes to this revision.Mar 7 2023, 18:13

Some more edits of wording.

I'd also consider removing the breaks in the the upgrade item paragraphs.

upgrade.html
16 ↗(On Diff #38332)
18 ↗(On Diff #38332)
20 ↗(On Diff #38332)
22 ↗(On Diff #38332)
This revision now requires changes to proceed.Mar 7 2023, 18:13

Improve wording, remove line breaks

Words look good now.

One last thing is to improve the image, but we don't need to block on that.

This revision is now accepted and ready to land.Mar 8 2023, 03:17
This revision was automatically updated to reflect the committed changes.
aimal added inline comments.
upgrade.html
18 ↗(On Diff #38341)

Typo: missing "be" in here.