Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move various e2e defintions out to yaml files #1166

Merged
merged 1 commit into from Jul 19, 2022

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Jul 12, 2022

We have code to generate tables, which we should use in the e2e section.

Preview: https://pr1166--matrix-spec-previews.netlify.app

@richvdh richvdh requested a review from a team as a code owner July 12, 2022 16:26
We have code to generate tables, which we should use in the e2e section.
@richvdh richvdh force-pushed the rav/clean_up_e2e_definitions branch from 964cf6c to f8e3220 Compare July 12, 2022 16:27
@@ -323,7 +323,6 @@ footer {
table {
table-layout: fixed;
width: 100%;
margin: 4rem 0;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did this because otherwise I ended up with a huge gap of whitespace above the new tables. I can't find anywhere else that it has changed anything.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the 200 response of https://pr1166--matrix-spec-previews.netlify.app/client-server-api/#getwell-knownmatrixclient for an example where this visually merges two adjacent tables one visual blob.

(xref: ea42cd3#r79081346)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here's a quick prototype. WDYT?

image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, lgtm!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

new PR incoming!

type: string
description: |-
The curve25519 public key used to encrypt the backups, encoded in unpadded base64.
example: "abcdefg"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +46 to +56
example: {
"algorithm": "m.megolm.v1.aes-sha2",
"forwarding_curve25519_key_chain": [
"hPQNcabIABgGnx3/ACv/jmMmiQHoeFfuLB17tzWp6Hw"
],
"sender_key": "RF3s+E7RkTQTGF2d8Deol0FkQvgII2aJDf3/Jp5mxVU",
"sender_claimed_keys": {
"ed25519": "aj40p+aw64yPIdsxoog8jhPu9i7l7NcFRecuOQblE3Y",
},
"session_key": "AgAAAADxKHa9uFxcXzwYoNueL5Xqi69IkD4sni8Llf..."
}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

examples in this file were based on the old example for the export format, but the ed25519 key has been replaced with something realistic as the angle brackets in the old example didn't render correctly.

algorithm:
type: string
description: |-
The end-to-end message encryption algorithm that the key is for. Must be `m.megolm.v1.aes-sha2`.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Descriptions in this file have been inspired by the changes in #1088

Comment on lines +21 to +23
{{ if not $definition }}
{{ errorf "site data %s not found" $path }}
{{ end }}
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I spent ages wondering why I was getting an obscure error. Turns out I had the filename wrong...

Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks :)

sorry for not getting to it sooner :(

@richvdh richvdh merged commit ea42cd3 into main Jul 19, 2022
@richvdh richvdh deleted the rav/clean_up_e2e_definitions branch July 19, 2022 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants