Commit graph

20245 commits

Author SHA1 Message Date
Robert Wolff 574265e4a8 fix golangci-lint regexpMust 2024-07-28 14:18:34 +00:00
Robert Wolff 58c60ec11c add issue checklist integration test 2024-07-28 14:18:34 +00:00
Robert Wolff b2bbf48c18 simplify regex for checkbox detection 2024-07-28 14:18:34 +00:00
Robert Wolff 9f461e180a remove requirement on trailing character after check box 2024-07-28 14:18:34 +00:00
Robert Wolff a96b86778f fix unchecking check boxes like [X] 2024-07-28 14:18:34 +00:00
Robert Wolff b222c48060 fix counting of all / done issue tasks, fix #4431 2024-07-28 14:18:34 +00:00
Earl Warren 22de4ae9c4 Merge pull request 'Update dependency @vitejs/plugin-vue to v5.1.1 (forgejo)' (#4713) from renovate/forgejo-vitejs-plugin-vue-5.x into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4713
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
2024-07-28 05:04:03 +00:00
Renovate Bot bd38b15e74 Update dependency @vitejs/plugin-vue to v5.1.1 2024-07-28 00:02:00 +00:00
Gusted 4c40bf5d29 Merge pull request '[SEC] Notify owner about TOTP enrollment' (#4704) from gusted/sec-more-totp into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4704
Reviewed-by: 0ko <0ko@noreply.codeberg.org>
2024-07-27 20:53:47 +00:00
Earl Warren e7a12286dd Merge pull request 'feat: optimize the FindUnreferencedPackages package query' (#4711) from earl-warren/forgejo:wip-container-simplify into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4711
Reviewed-by: thefox <thefox@noreply.codeberg.org>
2024-07-27 15:33:13 +00:00
Earl Warren ff6aceaeac
feat: optimize the FindUnreferencedPackages package query
Replace a double select with a simple select.

The complication originates from the initial implementation which
deleted packages instead of selecting them. It was justified to
workaround a problem in MySQL. But it is just a waste of resources
when collecting a list of IDs.
2024-07-27 15:57:24 +02:00
0ko dac78913aa i18n(en): new repo form: remove full stops from placeholders, clarify labels (#4709)
Just a small PR fixing some lines.

* removed full stop from placeholders, they were placed inconsistent and it doesn't really makes sense to have them in placeholders. It's usually either no full stop or ellipsis.
* s/Issue labels/Labels - obviously labels aren't used just for issues

That's about it. I may or may not send other improvements for this page later, this is just a simple PR that doesn't need testing.

Preview: https://codeberg.org/attachments/f7040bb4-9749-4cd2-9953-289e1103ea3e

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4709
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
2024-07-27 13:18:40 +00:00
Earl Warren 5d40264f0f Merge pull request 'Migrate renovate config' (#4708) from renovate/migrate-config into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4708
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
2024-07-27 06:51:17 +00:00
Renovate Bot a74f3832a5 Migrate config renovate.json 2024-07-27 06:09:28 +00:00
Earl Warren d9c82089ee Merge pull request 'Update renovate to v38 (forgejo) (major)' (#4703) from renovate/forgejo-major-renovate into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4703
Reviewed-by: Michael Kriese <michael.kriese@gmx.de>
2024-07-27 04:45:24 +00:00
Renovate Bot 5da7b3ae1f Update renovate to v38 2024-07-26 20:01:43 +00:00
Gusted a7e96aae66
[SEC] Notify owner about TOTP enrollment
- In the spirit of #4635
- Notify the owner when their account is getting enrolled into TOTP. The
message is changed according if they have security keys or not.
- Integration test added.
2024-07-26 19:49:22 +02:00
Gusted 8db2534082 Merge pull request '[MAIL] Use correct function in mail footer' (#4702) from gusted/forgejo-fix-auth-mail into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4702
Reviewed-by: 0ko <0ko@noreply.codeberg.org>
2024-07-26 14:11:30 +00:00
Gusted 685a903cd5
[MAIL] Use correct function in mail footer
- Regression of #4635
- The authentication mails weren't being sent with links to the
instance, because the the wrong variable was used in the mail footer.
`$.AppUrl` should've been `AppUrl`.
- Unit test added.
2024-07-26 15:16:54 +02:00
Earl Warren 2285526e93 Merge pull request 'chore(renovate): fix renovate grouping' (#4700) from viceice/renovate into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4700
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
2024-07-26 12:46:49 +00:00
Michael Kriese bc259542b5
chore(renovate): fix renovate grouping 2024-07-26 14:07:39 +02:00
Earl Warren 904925ee4d Merge pull request 'Update dependency tailwindcss to v3.4.7 (forgejo)' (#4695) from renovate/forgejo-tailwindcss-3.x into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4695
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
2024-07-26 04:52:59 +00:00
Renovate Bot c192e70b94 Update dependency tailwindcss to v3.4.7 2024-07-26 00:02:10 +00:00
Earl Warren 1f939a2a40 Merge pull request 'feat(release-notes-assistant): diff of the change in preview' (#4693) from twenty-panda/forgejo:wip-rna into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4693
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
2024-07-25 20:51:16 +00:00
Earl Warren 90f80be3f0 Merge pull request 'Fix conversation template' (#4691) from 0ko/forgejo:ui-roles-reviews into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4691
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
2024-07-25 20:46:11 +00:00
Twenty Panda b85ec067ca feat(release-notes-assistant): diff of the change in preview 2024-07-25 22:32:14 +02:00
0ko 13cb4b3f2e fix(ui): fix conversation template 2024-07-25 23:51:17 +05:00
Earl Warren edf2cbe768 Merge pull request '[I18N] Remove unneeded translation' (#4690) from gusted/forgejo-rm-unneeded-tr into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4690
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
2024-07-25 16:04:49 +00:00
Gusted 76759715d2
[I18N] Remove unneeded translation
- This key isn't used, I added this on accident (likely bad copy-paste).
- Regression of #4635
2024-07-25 17:04:23 +02:00
Earl Warren c43bb32010 Merge pull request 'fix(api): issue state change is not idempotent' (#4687) from earl-warren/forgejo:wip-issue-state-idempotency into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4687
Reviewed-by: Michael Kriese <michael.kriese@gmx.de>
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
2024-07-25 14:20:34 +00:00
Earl Warren e9e3b8c0f3
fix(api): issue state change is not idempotent
The PATCH if issue & pull request switched to use the service
functions instead. However, the service function changing the state is
not idempotent. Instead of doing nothing which changing from open to
open or close to close, it will fail with an error like:

 Issue [2472] 0 was already closed

Regression of: 6a4bc0289d

Fixes: https://codeberg.org/forgejo/forgejo/issues/4686
2024-07-25 15:16:44 +02:00
Earl Warren f8bc48f8df Merge pull request 'Update dependency vue to v3.4.34 (forgejo)' (#4678) from renovate/forgejo-patch-vue-monorepo into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4678
Reviewed-by: Michael Kriese <michael.kriese@gmx.de>
2024-07-25 08:50:10 +00:00
Earl Warren 8571f1d5d9 Merge pull request 'Update dependency monaco-editor to v0.50.0 (forgejo)' (#4604) from renovate/forgejo-monaco-editor-0.x into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4604
Reviewed-by: Michael Kriese <michael.kriese@gmx.de>
2024-07-25 08:10:30 +00:00
Earl Warren 0713451c28 Merge pull request 'Update dependency postcss to v8.4.40 (forgejo)' (#4677) from renovate/forgejo-postcss-packages into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4677
Reviewed-by: Michael Kriese <michael.kriese@gmx.de>
2024-07-25 08:10:20 +00:00
Renovate Bot 03c1b61573 Update dependency vue to v3.4.34 2024-07-25 08:04:49 +00:00
Earl Warren 875e6496c6 Merge pull request '[CHORE] Remove AGPL-1.0 as allowed license' (#4673) from gusted/forgejo-rm-agpl into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4673
Reviewed-by: Beowulf <beowulf@noreply.codeberg.org>
2024-07-25 07:40:19 +00:00
Earl Warren 783fa3e108 Merge pull request 'fix(release-notes-assistant): remove extra newline on breaking bugs' (#4683) from earl-warren/forgejo:wip-rna-preview into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4683
Reviewed-by: twenty-panda <twenty-panda@noreply.codeberg.org>
2024-07-25 07:32:51 +00:00
Earl Warren 4f7746427f
fix(release-notes-assistant): remove extra newline on breaking bugs 2024-07-25 08:26:03 +02:00
Robert Wolff 0a74c95b2a fix(ui): use right placeholder string in milestones search (#4628)
This PR fixes a wrong placeholder for the search for milestones. I tested it locally (see attachments, below).

Before: https://codeberg.org/attachments/ba845ce1-1f20-4131-a74d-7220986a4acf
After: https://codeberg.org/attachments/0c4e32ee-b1a8-4472-837d-daa2a2a50121

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4628
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
Reviewed-by: 0ko <0ko@noreply.codeberg.org>
Co-authored-by: Robert Wolff <mahlzahn@posteo.de>
Co-committed-by: Robert Wolff <mahlzahn@posteo.de>
2024-07-25 03:33:44 +00:00
Gusted 56dbe3263f Merge pull request '[CHORE] Don't bundle elkjs' (#4670) from gusted/forgejo-elkjs into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4670
Reviewed-by: Beowulf <beowulf@noreply.codeberg.org>
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
2024-07-25 00:15:35 +00:00
Renovate Bot c23cac5073 Update dependency postcss to v8.4.40 2024-07-25 00:04:25 +00:00
Gusted a6b4fb7729
[CHORE] Remove AGPL-1.0 as allowed license
- The last dependency using this license was removed in #4595 (thanks @Beowulf!).
2024-07-24 23:38:31 +02:00
Earl Warren 419c129ec7 Merge pull request 'Replace Gitea with Forgejo' (#4660) from yonas/forgejo:yonas-patch-1 into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4660
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
2024-07-24 21:35:14 +00:00
Earl Warren c75c6b392c Merge pull request '[UI] Fix scoped-access-token' (#4669) from gusted/forgejo-fix-regg into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4669
Reviewed-by: Beowulf <beowulf@noreply.codeberg.org>
2024-07-24 21:34:19 +00:00
Gusted 510cbe2c92
[CHORE] Don't bundle elkjs
- `elkjs` is a library that's imported by `mermaid`, although they have
seperated this package to it's own mermaid
package (https://github.com/mermaid-js/mermaid/pull/5654), the stable
version doesn't have this patch.
- `elkjs` is licensed under the EPL-2.0 license (copyleft), which isn't
compatible with GPL unless the license author explcitly allow this via a
so called "secondary license". At the end of the day it cannot be
released under a MIT or GPL license.
- Use webpack's `externals` option to avoid bundling `elkjs` and instead
leave it as a `require` code.
- This is a 'dirty' way to ensure elkjs isn't bundled and has to be
tested manually to ensure this for every release (via the
`webpack-bundle-analyzer` plugin). If someone tries to use the elkjs
render, it will result in a non-descriptive error being shown.
2024-07-24 23:28:59 +02:00
Gusted aa5163d5c4
[UI] Fix scoped-access-token
- Regression of #4571
- Refactor to not use the component API of Vue. The root cause is still
unknown.
2024-07-24 21:48:51 +02:00
Earl Warren b55136d561 Merge pull request 'fix(release-notes-assistant): do not trigger on open' (#4665) from twenty-panda/forgejo:wip-rna-trigger into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4665
2024-07-24 16:32:43 +00:00
Twenty Panda e9a3306f6e fix(release-notes-assistant): do not trigger on open
Forgejo sets a label and will notify this when opening the pull
request. Triggering when it opens will make two workflows for the same
SHA. Re-opening is a border case that is not needed.
2024-07-24 18:26:59 +02:00
Earl Warren 762f4b5408 Merge pull request 'fix(release-notes-assistant): ignore WIP prefixes' (#4663) from earl-warren/forgejo:wip-rna-preview into forgejo
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4663
Reviewed-by: twenty-panda <twenty-panda@noreply.codeberg.org>
2024-07-24 14:55:29 +00:00
Earl Warren ba006b2eef
fix(release-notes-assistant): ignore WIP prefixes 2024-07-24 16:49:21 +02:00