summaryrefslogtreecommitdiff
path: root/packages (follow)
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
* 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-043-5/+11
|
* update summalysyuilo2022-06-042-5/+6
|
* Revert "feat: option to collapse long notes (#8561)"syuilo2022-06-041-24/+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-041-3/+24
| | | | | | | | | | | | | * 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-0311-88/+31
|\
| * fix(client): correctly handle MiAuth URLs with query string (#8772)sn0w2022-06-031-1/+4
| |
| * 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
| |
| * fix: always remove completed tasks (#8771)Johann1502022-05-312-1/+4
| |
| * Fix IP address rate limit (#8758)MeiMei2022-05-313-9/+14
| | | | | | | | | | | | | | * Fix IP address rate limit * CHANGELOG * Tune getIpHash
| * chore: fix import tinycolortamaina2022-05-301-1/+1
| |
| * Merge branch 'develop' of https://github.com/misskey-dev/misskey into developtamaina2022-05-30219-5287/+4850
| |\
| * | 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
* | fix(client): fix popout url (#8494)futchitwo2022-05-292-3/+4
| |
* | refactor: improve code quality (#8751)Johann1502022-05-296-77/+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-2815-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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
* | enhance: replace signin CAPTCHA with rate limit (#8740)Johann1502022-05-285-57/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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
* | fix(docs): correct information for drive upload (#8736)Johann1502022-05-271-2/+14
| |
* | fix lints (#8737)Johann1502022-05-2669-257/+255
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix: emits use ev instead of e * fix: errors use err instead of e * fix: replace use of data where possible * fix: events use evt instead of e * fix: use strict equals * fix: use emoji instead of e * fix: vue lints
* | use http-signature module that supports hs2019 (#8635)Johann1502022-05-267-19/+19
| |
* | update depssyuilo2022-05-254-732/+608
| |
* | refactor: use ===syuilo2022-05-251-2/+2
| |
* | fix: assume remote users are following each other (#8734)Johann1502022-05-251-9/+15
| | | | | | | | | | | | Misskey does not know if two remote users are following each other. Because ActivityPub actions would otherwise fail on followers only notes, we have to assume that two remote users are following each other when an interaction about a remote note occurs.
* | fix: activity widget used wrong variable nameJohann1502022-05-251-3/+3
| |
* | fix: server metrics widgetJohann1502022-05-251-4/+4
| |
* | refactor: temporary files (#8713)Johann1502022-05-2512-331/+297
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * simplify temporary files for thumbnails Because only a single file will be written to the directory, creating a separate directory seems unnecessary. If only a temporary file is created, the code from `createTemp` can be reused here as well. * refactor: deduplicate code for temporary files/directories To follow the DRY principle, the same code should not be duplicated across different files. Instead an already existing function is used. Because temporary directories are also create in multiple locations, a function for this is also newly added to reduce duplication. * fix: clean up identicon temp files The temporary files for identicons are not reused and can be deleted after they are fully read. This condition is met when the stream is closed and so the file can be cleaned up using the events API of the stream. * fix: ensure cleanup is called when download fails * fix: ensure cleanup is called in error conditions This covers import/export queue jobs and is mostly just wrapping all code in a try...finally statement where the finally runs the cleanup. * fix: use correct type instead of `any`
* | Refactor widgets and fix lint issues (#8719)Andreas Nedbal2022-05-2526-313/+256
| | | | | | | | | | | | | | | | | | * fix(client): refactor widgets and fix lint issues * Apply review suggestions from @Johann150 Co-authored-by: Johann150 <johann@qwertqwefsday.eu> Co-authored-by: Johann150 <johann@qwertqwefsday.eu>
* | refactor(client): refactor admin/other-settings to use Composition API (#8667)Andreas Nedbal2022-05-251-39/+22
| |
* | Refactor admin/overview to use Composition API (#8674)Andreas Nedbal2022-05-251-79/+37
| | | | | | | | | | * refactor(client): refactor admin/overview to use Composition API * fix(client): apply review suggestions