Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

[meta] Good First Issue watchtower #9486

Closed
ShadowJonathan opened this issue Feb 24, 2021 · 11 comments
Closed

[meta] Good First Issue watchtower #9486

ShadowJonathan opened this issue Feb 24, 2021 · 11 comments

Comments

@ShadowJonathan
Copy link
Contributor

ShadowJonathan commented Feb 24, 2021

Context:

<@anoadragon453> If you're up for it, and come across some that are perhaps too easy, or you just don't want to solve, would you mind flagging them and I can label them Good First Issue?

This is that issue, I'll add them in the following format:

issue: #XXXX
e.s.t.: P3, T-Enhancement, (etc.)
comment: (Here is some extra comments and context to go with)

(e.s.t./est is "extra suggested tags")

@ShadowJonathan
Copy link
Contributor Author

ShadowJonathan commented Feb 24, 2021

issue: #4173
e.s.t.: P3, T-Enhancement
comment: Seems to simplify the Element clients, and i've encountered some situations where this doesn't properly happen, I'm pretty sure there's a way to instantly update account data from a room creation handler.

@ShadowJonathan
Copy link
Contributor Author

issue: #9340
e.s.t.: (already triaged)
comment: Seems to be simple enough to add a if txn_id == "" check somewhere.

@ShadowJonathan
Copy link
Contributor Author

issue: #8551
e.s.t.: P4 (or P3), T-Task
comment: Seems to be clear enough for someone to poke at the codebase and implement it :P

@ShadowJonathan
Copy link
Contributor Author

issue: #5538
est: P4, T-Task
comment: Seems to be simple enough, riot-element is mentioned, but imma assume that was Riot before RiotX, maybe it's also helpful to inspect element-android before making that issue a go-ahead.

@ShadowJonathan
Copy link
Contributor Author

issue: #4175
est: S-Tolerable, P3, T-Task
comment: Maybe not 100% a good issue for anyone completely new to matrix, needs a bit of knowledge on how to construct a mxc uri, and some hashing out would probably need to be done in PR review to use the right helper methods (if these exist)

@ShadowJonathan
Copy link
Contributor Author

(tentative)
issue: #4363
est: S-Minor, P3, T-Task or T-Defect
comment: I'm not entirely sure about this one, it seems to be a bit too complex for a good first issue, i'll take up on it if it isn't.

@ShadowJonathan
Copy link
Contributor Author

ShadowJonathan commented Feb 24, 2021

issue: #3058
est: P3, T-Task
comment: This seems to mention synapse v0.26, is it possible this is already fixed? Maybe helpful to look at that first. Other than that, it looks to be a simple if "to" not in query.keys() check :P

@ShadowJonathan
Copy link
Contributor Author

issue: #9032
est: P3, T-Task
comment: This seems already claimed by @PaarthShah, but it's been longer than a week since his reply, so i'm putting it on here for probable GFI triaging :)

@ShadowJonathan
Copy link
Contributor Author

(tentative)
issue: #6085
est: P4, T-Task
comment: Not really sure about this, because this means changing pure-static files to one-time templated ones, which might be difficult to do, as well as with other solutions.

@ShadowJonathan
Copy link
Contributor Author

ShadowJonathan commented Feb 24, 2021

issue: #1521 (closed)
est: P4, T-Task, S-Tolerable
comment: Seems to be easy enough to add a proper check for a MXID, and then fail early with 400 on that.

@richvdh
Copy link
Member

richvdh commented May 24, 2021

I think this issue has served its purpose

@richvdh richvdh closed this as completed May 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants