Skip to content

Future Events: Add future endpoints to ruma.#1845

Merged
zecakeh merged 6 commits intoruma:mainfrom
toger5:toger5/future-requests
Jun 25, 2024
Merged

Future Events: Add future endpoints to ruma.#1845
zecakeh merged 6 commits intoruma:mainfrom
toger5:toger5/future-requests

Conversation

@toger5
Copy link
Contributor

@toger5 toger5 commented Jun 18, 2024

Relies on: #1848

This implements the required endpoint for MSC4140.

This is the first step to make the widget api support Futures and will be used in the rust-sdk widget driver.

@toger5 toger5 changed the title Add future endpoints to ruma. Future Events: Add future endpoints to ruma. Jun 18, 2024
@toger5 toger5 force-pushed the toger5/future-requests branch 5 times, most recently from 52dbef8 to 9ae13f0 Compare June 19, 2024 11:15
@toger5 toger5 marked this pull request as ready for review June 19, 2024 11:15
@toger5 toger5 force-pushed the toger5/future-requests branch 2 times, most recently from 76607d6 to 9a08459 Compare June 19, 2024 11:26
Copy link
Contributor

@zecakeh zecakeh left a comment

Choose a reason for hiding this comment

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

I tried to do a thorough review but avoid to repeat the same comments if it seemed like the code was copy-pasted, so some comments on docs might need to be fixed in other places too.

@toger5 toger5 force-pushed the toger5/future-requests branch from 11a2c3b to ccdd564 Compare June 19, 2024 14:52
@toger5 toger5 force-pushed the toger5/future-requests branch from d722405 to 561a516 Compare June 20, 2024 09:59
Copy link
Contributor

@zecakeh zecakeh left a comment

Choose a reason for hiding this comment

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

This looks good, however we need to merge #1848 first.

@zecakeh
Copy link
Contributor

zecakeh commented Jun 21, 2024

Alright, if you rebase on main we should be good to go.

@toger5 toger5 force-pushed the toger5/future-requests branch from 8fca79e to 853197a Compare June 23, 2024 14:20
@toger5 toger5 force-pushed the toger5/future-requests branch from 28b5c6f to 6a2ab00 Compare June 23, 2024 14:25
@toger5 toger5 requested a review from zecakeh June 23, 2024 14:30
@zecakeh
Copy link
Contributor

zecakeh commented Jun 24, 2024

Ah we are just missing a changelog entry now.

@toger5
Copy link
Contributor Author

toger5 commented Jun 24, 2024

I added a change-log entry.

@toger5 toger5 requested a review from zecakeh June 24, 2024 14:25
@toger5 toger5 force-pushed the toger5/future-requests branch from 1fec53d to 6515128 Compare June 25, 2024 09:38
@toger5 toger5 requested a review from zecakeh June 25, 2024 09:38
@toger5 toger5 force-pushed the toger5/future-requests branch from 6515128 to 3befe56 Compare June 25, 2024 10:02
@zecakeh zecakeh merged commit 862be07 into ruma:main Jun 25, 2024
@zecakeh
Copy link
Contributor

zecakeh commented Jun 25, 2024

Thanks for this!

@toger5 toger5 deleted the toger5/future-requests branch July 5, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants