matrix-spec-proposals icon indicating copy to clipboard operation
matrix-spec-proposals copied to clipboard

MSC3715: Add a pagination direction parameter to `/relations`

Open clokep opened this issue 3 years ago • 11 comments
trafficstars

Rendered

Implementation: matrix-org/synapse#11941.

FCP Call

clokep avatar Feb 07 '22 19:02 clokep

@clokep says: Hello! 👋 I believe that both MSC3666 and MSC3715 are ready for consideration by the SCT. They're both minor improvements that build on MSC2675.

richvdh avatar Feb 08 '22 15:02 richvdh

@mscbot fcp merge

erikjohnston avatar Mar 08 '22 16:03 erikjohnston

Team member @mscbot has proposed to merge this. The next step is review by the rest of the tagged people:

  • [x] @dbkr
  • [x] @uhoreg
  • [x] @turt2live
  • [x] @ara4n
  • [ ] @anoadragon453
  • [x] @richvdh
  • [x] @erikjohnston
  • [x] @KitsuneRal

Concerns:

  • ~~unclear why this is required~~
  • ~~lacking a client implementation~~

Once at least 75% of reviewers approve (and there are no outstanding concerns), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for information about what commands tagged team members can give me.

mscbot avatar Mar 08 '22 16:03 mscbot

@mscbot concern unclear why this is required

richvdh avatar Mar 22 '22 16:03 richvdh

@mscbot resolve concern unclear why this is required

richvdh avatar Jun 08 '22 21:06 richvdh

@mscbot resolve concern unclear why this is required

richvdh avatar Jun 08 '22 21:06 richvdh

Unknown concern 'concern unclear why this is required'.

mscbot avatar Jun 08 '22 21:06 mscbot

Unknown concern 'concern unclear why this is required'.

mscbot avatar Jun 08 '22 21:06 mscbot

@mscbot resolve unclear why this is required

richvdh avatar Jun 08 '22 21:06 richvdh

There seems to be a server implementation referenced, which is good, but it's unclear how or if this is useful to clients if no one has implemented it.

@mscbot concern lacking a client implementation

turt2live avatar Jul 26 '22 14:07 turt2live

There seems to be a server implementation referenced, which is good, but it's unclear how or if this is useful to clients if no one has implemented it.

matrix-ios-sdk attempts to use it, but ends up not using the proper parameter. So that likely doesn't count as an implementation.

clokep avatar Jul 27 '22 16:07 clokep

Status of this MSC: element-web has recently picked up activity on https://github.com/vector-im/element-web/issues/22501 to see if this MSC is needed, and how to fix it if so.

turt2live avatar Sep 06 '22 19:09 turt2live

@mscbot resolve lacking a client implementation

turt2live avatar Sep 13 '22 16:09 turt2live

:bell: This is now entering its final comment period, as per the review above. :bell:

mscbot avatar Sep 13 '22 16:09 mscbot

The final comment period, with a disposition to merge, as per the review above, is now complete.

mscbot avatar Sep 18 '22 16:09 mscbot

Spec PR: https://github.com/matrix-org/matrix-spec/pull/1254

turt2live avatar Sep 27 '22 04:09 turt2live

Merged 🎉

turt2live avatar Sep 27 '22 19:09 turt2live