celestia-app icon indicating copy to clipboard operation
celestia-app copied to clipboard

Tracking: Add an official spec for the app

Open evan-forbes opened this issue 1 year ago • 6 comments

While archived, the specs contain a lot of really important and critical designs that are effectively cannon. We should move as much as we can over from them, while refactoring, modifying, and updating to match the current implementation.

some special love is needed for our the encoding section, as its much more difficult to decipher the share splitting/merging implementation

  • [ ] #431
  • [ ] #688
  • [ ] #727
  • [ ] #728
  • [ ] #729
  • [ ] #730
  • [ ] #731
  • [ ] #732
  • [x] #733
  • [ ] #734
  • [ ] #735
  • [ ] #736
  • [ ] #737
  • [ ] #738
  • [ ] #739
  • [ ] #743
  • [ ] #751
  • [ ] #784
  • [x] #792
  • [ ] #795
  • [x] https://github.com/celestiaorg/celestia-core/issues/866

evan-forbes avatar Aug 23 '22 02:08 evan-forbes

One alternative approach is to unarchive the specs repo and remove the outdated specs.

rootulp avatar Aug 23 '22 05:08 rootulp

Per-repo specs will be easier to maintain, as each team can update spec in tandem with implementation changes.

adlerjohn avatar Aug 23 '22 13:08 adlerjohn

I'm going to use this comment to continually add reminders as I think of them. Others should feel free to edit this comment or post their own comment as well.

we introduced a new validity rule here for MsgPayForData and MsgWirePayForData. https://github.com/celestiaorg/celestia-app/pull/666

evan-forbes avatar Aug 31 '22 16:08 evan-forbes

Relevant: https://github.com/celestiaorg/celestia-app/issues/727

rootulp avatar Aug 31 '22 16:08 rootulp

nice!! I didn't see that

evan-forbes avatar Aug 31 '22 17:08 evan-forbes

Can we turn this into a tracking issues @evan-forbes

adlerjohn avatar Sep 19 '22 19:09 adlerjohn

The "Pure specs" issues are blocking the merge of specs-staging into main.

rootulp avatar Nov 01 '22 15:11 rootulp