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.

Sign up for GitHub

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

Jump to bottom

Switch /timestamp_to_event endpoint to stable v1 #142 #154

Merged
merged 1 commit into from
Feb 17, 2023
Merged

Switch /timestamp_to_event endpoint to stable v1 #142 #154

merged 1 commit into from
Feb 17, 2023

Conversation

pdurbin
Copy link
Contributor

@pdurbin pdurbin commented Dec 20, 2022

Relates to this issue:

Perhaps some of the README edits (and a default config comment) can be backed out or reworded that are part of this PR...

... but I'll leave it to someone who can word it better.

@MadLittleMods MadLittleMods changed the title switch timestamp_to_event to v1 #142 Switch /timestamp_to_event endpoint to stable v1 #142 Dec 20, 2022
@MadLittleMods MadLittleMods added T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. community-contribution labels Dec 20, 2022
@bil-ash
Copy link

bil-ash commented Feb 11, 2023

Any update on when this will be merged?

@MadLittleMods MadLittleMods merged commit 8f9e163 into matrix-org:main Feb 17, 2023
MadLittleMods added a commit that referenced this pull request Feb 17, 2023
@MadLittleMods
Update docs to reflect that we need stable MSC3030 /timstamp_to_event
3cde7d8 
Follow-up to #154
@MadLittleMods MadLittleMods mentioned this pull request Feb 17, 2023
Merged
@MadLittleMods
Copy link
Contributor

Thanks for updating this @pdurbin!

I've followed this up with some doc updates as you mentioned in the description 👍 -> #159

MadLittleMods added a commit that referenced this pull request Feb 17, 2023
@MadLittleMods
Update docs to reflect that we need stable MSC3030 /timstamp_to_event (…
b0cee80 
…#159)

Follow-up to #154
@pdurbin pdurbin deleted the timestamp_to_event branch February 17, 2023 23:37
@pdurbin pdurbin mentioned this pull request Mar 8, 2023
Closed
@MadLittleMods MadLittleMods added the A-backend Related to the backend Node.js server pieces label Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MadLittleMods MadLittleMods MadLittleMods left review comments

Assignees
No one assigned
Labels
A-backend Related to the backend Node.js server pieces community-contribution T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pdurbin @bil-ash @MadLittleMods