Cliff Hall

Change case on return value & format code for readability

1

https://github.com/MaxflowO2/ERC2981/blob/010744cac3e6c5214a184733faa59cca61884daa/contracts/ERC2981.sol#L49 Readability is important in reference and framework code. Formatting it with that in mind (as we did in the ERC), will be appreciated by those who evaluate it when...

Comment folding is broken

6

_Using the latest Webstorm IDE with latest Solidity plugin 2.3.6._ What is going on with this comment folding? It seems to think we need control points at the NatSpec identifiers:...

Can't command-click into inherited members

1

_Using latest version of Webstorm and Solidity plugin 2.3.6._ Example: In this example, the function `_setupRole` is inherited from `AccessControl.sol`. I expect that cmd-click on `_setupRole` will take me to...

Support user owned NFTs as avatars

4

Ethereum users who collect NFTs are increasingly interested in using them for identity purposes. For instance:  https://twitter.com/ethlance/status/1234730022063280128?s=21 I am the developer of the Avastars project mentioned in that tweet,...

X-axis labels appear and disappear on scroll

1

The x-axis labels appear and disappear when you scroll left & right. Note that the issue only happens when you're zoomed out. When you're zoomed in it doesn't appear to...

Would be nice to know this up front in the docs.

slides-per-view="auto" doesn't work

2

```html angular.module('app', ['ui.swiper']); ``` With this basic setup using http://idangero.us/swiper you get a single row of items, you can drag them, and you'll see the new ones enter at the...

Normalize reporting of failed connection.

3

_Originally posted by @halter73 in https://github.com/modelcontextprotocol/inspector/pull/339#pullrequestreview-2789087665_ > I tested this with the sample Streamable HTTP server at https://github.com/modelcontextprotocol/csharp-sdk/tree/v0.1.0-preview.11/samples/AspNetCoreSseServer (it has yet to be renamed from "Sse"), and things seem to...

Send HTTP DELETE on streamable transport disconnect

1

* In server/src/index.ts - add delete handler for /mcp endpoint - gets the server transport for the sessionId - calls terminateSession on it - removes the webapp and server transports...

## DON'T MERGE UNTIL #470 IS MERGED * In server/index.js - in createTransport() - stringify query params to put them on one line - remove logging of transport creation (done...