summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* refactor: use awaitAll to reduce duplication (#8791)Johann1502022-06-081-61/+25
| | | | | | | * refactor: use awaitAll to reduce duplication * fix lint * fix typo
* Update .node-versionsyuilo2022-06-061-1/+1
|
* fix(test): make chart tests workingsyuilo2022-06-053-53/+65
|
* Update .mocharc.jsonsyuilo2022-06-051-1/+1
|
* chore: lint fixessyuilo2022-06-057-40/+41
|
* chore(client): fix menu item stylesyuilo2022-06-051-10/+11
|
* use node 16syuilo2022-06-045-9/+12
|
* update summalysyuilo2022-06-042-5/+6
|
* Update CHANGELOG.mdsyuilo2022-06-041-1/+0
|
* Update CONTRIBUTING.mdsyuilo2022-06-041-2/+2
|
* Update CONTRIBUTING.mdsyuilo2022-06-041-6/+8
|
* Update CHANGELOG.mdsyuilo2022-06-041-15/+29
|
* Revert "feat: option to collapse long notes (#8561)"syuilo2022-06-042-25/+3
| | | | This reverts commit e675ffcf38b07f5c70d00b49c171c7ab3460e810.
* fix bugsyuilo2022-06-041-3/+3
|
* fix testsyuilo2022-06-041-1/+3
|
* feat: option to collapse long notes (#8561)Balazs Nadasdi2022-06-042-3/+25
| | | | | | | | | | | | | * feat: option to collapse long notes Closes #8559 * do not collapse if cw exists * use '閉じる' to close / show less. * make it sticky * Change style of the Show less button
* fix: add id for activitypub follows (#8689)Johann1502022-06-045-15/+63
| | | | | | | | | | | | | | | | | * add id for activitypub follows * fix lint * fix: follower must be local, followee must be remote Misskey will only use ActivityPub follow requests for users that are local and are requesting to follow a remote user. This check is to ensure that this endpoint can not be used by other services or instances. * fix: missing import * render block with id * fix comment
* fix: ensure resolver does not fetch local resources via HTTP(S) (#8733)Johann1502022-06-042-50/+115
| | | | | | | | | | | | | | | | | | | | | * refactor: parseUri types and checks The type has been refined to better represent what it actually is. Uses of parseUri are now also checking the parsed object type before resolving. * cannot resolve URLs with fragments * also take remaining part of URL into account Needed for parsing the follows URIs. * Resolver uses DbResolver for local * remove unnecessary use of DbResolver Using DbResolver would mean that the URL is parsed and handled again. This duplicated processing can be avoided by querying the database directly. * fix missing property name
* fix: correctly render empty note text (#8746)Johann1502022-06-032-7/+5
| | | | | | | | | | | | | | | Ensure that the _misskey_content attribute will always exist. Because the API endpoint does not require the existence of the `text` field, that field may be `undefined`. By using `?? null` it can be ensured that the value is at least `null`. Furthermore, the rendered HTML of a note with empty text will also be the empty string. From git blame it seems that this behaviour was added because of a Mastodon bug that might have previously existed. Hoever, this seems to be no longer the case as I can find mastodon posts that have empty content. The code could be made a bit more succinct by using the null coercion operator.
* User moderation details (#8762)PikaDude2022-06-032-5/+40
| | | | | | | | | | | | | | | | | * add more user details for admins to see * fix some issues * small style fix as suggested by Johann150 Co-authored-by: Johann150 <johann@qwertqwefsday.eu> * fix Co-authored-by: Johann150 <johann@qwertqwefsday.eu> Co-authored-by: Johann150 <johann@qwertqwefsday.eu>
* Merge branch 'develop' of https://github.com/misskey-dev/misskey into developsyuilo2022-06-0318-167/+85
|\
| * fix(client): correctly handle MiAuth URLs with query string (#8772)sn0w2022-06-031-1/+4
| |
| * fix(dev): no labels for l10n_developJohann1502022-06-011-1/+3
| |
| * fix: server metrics widgetJohann1502022-06-011-4/+4
| |
| * fix(lint): indentationJohann1502022-05-311-1/+1
| |
| * fix(mfm): remove duplicate br tag/newline (#8616)Johann1502022-05-311-0/+3
| |
| * Extract commonly used test logic to commands (#8767)Andreas Nedbal2022-05-313-76/+44
| | | | | | | | | | | | | | | | | | * meta(tests): enable workflows to run in branch * feat(tests): move commonly used logic to Cypress commands * chore(tests): replace more code with commands * meta(tests): disable workflows to run in branch
| * Fix `Cannot find module` issue (#8770)MeiMei2022-05-312-1/+2
| | | | | | | | | | * Add --force to yarn in the installation script * CHAGELOG
| * fix: always remove completed tasks (#8771)Johann1502022-05-312-1/+4
| |
| * Fix IP address rate limit (#8758)MeiMei2022-05-314-10/+15
| | | | | | | | | | | | | | * Fix IP address rate limit * CHANGELOG * Tune getIpHash
| * chore(meta): label Pull Requests containing tests (#8768)Andreas Nedbal2022-05-311-0/+4
| |
| * chore: fix import tinycolortamaina2022-05-301-1/+1
| |
| * Merge branch 'develop' of https://github.com/misskey-dev/misskey into developtamaina2022-05-30266-5381/+7791
| |\
| * | chore: remove packages/sw/webpack.config.jstamaina2022-05-301-71/+0
| | |
* | | fix(test): reset redis in e2e testsyuilo2022-06-031-0/+2
| |/ |/| | | | | #7986
* | fix(client): import shared ESLint config in client package (#8761)Andreas Nedbal2022-05-302-47/+59
| |
* | fix: add missing importJohann1502022-05-291-1/+1
| | | | | | | | fix #8756
* | enhance(dev): ask for log snippetsJohann1502022-05-291-1/+4
| |
* | fix(client): fix popout url (#8494)futchitwo2022-05-292-3/+4
| |
* | refactor: improve code quality (#8751)Johann1502022-05-297-78/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * remove unnecessary if `Array.prototype.some` already returns a boolean so an if to return true or false is completely unnecessary in this case. * perf: use count instead of find When using `count` instead of `findOneBy`, the data is not unnecessarily loaded. * remove duplicate null check The variable is checked for null in the lines above and the function returns if so. Therefore, it can not be null at this point. * simplify `getJsonSchema` Because the assigned value is `null` and the used keys are only shallow, use of `nestedProperty.set` seems inappropriate. Because the value is not read, the initial for loop can be replaced by a `for..in` loop. Since all keys will be assigned `null`, the condition of the ternary expression in the nested function will always be true. Therefore the recursion case will never happen. With this the nested function can be eliminated. * remove duplicate condition The code above already checks `dragging` and returns if it is truthy. Checking it again later is therefore unnecessary. To make this more obvious the `return` is removed in favour of using an if...else construct. * remove impossible "unknown" time The `ago` variable will always be a number and all non-negative numbers are already covered by other cases, the negative case is handled with `future` so there is no case when `unkown` could be achieved.
* | fix(client): Vite related boot mechanism revision (#8753)tamaina2022-05-293-17/+14
| | | | | | | | | | | | | | | | | | * preload app css * remove salt * APP_FETCH_FAILED error * set max-age to 15s
* | preload app css (#8752)tamaina2022-05-293-15/+23
| |
* | chore: remove unused importsJohann1502022-05-281-3/+3
| |
* | fix: validate text is not emptyJohann1502022-05-281-1/+1
| | | | | | fix #8747
* | refactor: use css module at components/global/loading.vue (#8750)tamaina2022-05-291-31/+33
| | | | | | | | | | * refactor: use css module at components/global/loading.vue * rename class name to "root"
* | Fix theme import (#8749)tamaina2022-05-285-31/+46
| |
* | feat(tests): add e2e tests for widgets (#8735)Andreas Nedbal2022-05-2816-16/+100
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * test(e2e): add baseline for widget tests * chore(repo): enable test running in branch * fix(e2e): set viewport for widget tests * fix(client): add widget identifier classes to widgets * test(e2e): add memo widget test * fix(tests): force select value * fix(tests): force button press for widget addition * fix(tests): invoke select value differently * fix(tests): adjust widget submit * fix(tests): don't explicitly navigate for widget test * fix(tests): click label to hide select popup * fix(tests): just click modal background * fix(tests): adjust modal background selector * fix(tests): click all modal backgrounds * feat(e2e): add test for adding timeline widget * fix(client): add more widget identifier classes * feat(tests): add method abstraction for test cases * fix(tests): force-click overlays * fix(tests): force widget button press * fix(tests): remove timeout from final widget check * feat(tests): add widget removal test case * fix(client): use mk instead of msky as class prefix * fix(tests): check widgets for existence rather than visibility * chore(meta): don't run tests for specific feature branch
* | Remove require captcha from signin from CHANGELOG (#8748)Kainoa Kanter2022-05-281-1/+0
| |
* | enhance: replace signin CAPTCHA with rate limit (#8740)Johann1502022-05-287-57/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * enhance: rate limit works without signed in user * fix: make limit key required for limiter As before the fallback limiter key will be set from the endpoint name. * enhance: use limiter for signin * Revert "CAPTCHA求めるのは2fa認証が無効になっているときだけにした" This reverts commit 02a43a310f6ad0cc9e9beccc26e51ab5b339e15f. * Revert "feat: make captcha required when signin to improve security" This reverts commit b21b0580058c14532ff3f4033e2a9147643bfca6. * fix undefined reference * fix: better error message * enhance: only handle prefix of IPv6
* | enhance: clearly link documentationJohann1502022-05-271-0/+4
| | | | | | fix #8744