Skip to content

Message for closed registration #8462

Merged
Alkarex merged 9 commits intoFreshRSS:edgefrom
rupakbajgain:new-message
Jan 24, 2026
Merged

Message for closed registration #8462
Alkarex merged 9 commits intoFreshRSS:edgefrom
rupakbajgain:new-message

Conversation

@rupakbajgain
Copy link
Contributor

Closes #8269

Changes proposed in this pull request:

image image

Pull request checklist:

  • clear commit messages
  • code manually tested

@rupakbajgain
Copy link
Contributor Author

I think this is the last issue i could work on easily, other look very domain specific.

@Alkarex Alkarex added this to the 1.28.1 milestone Jan 22, 2026
@Alkarex
Copy link
Member

Alkarex commented Jan 22, 2026

Thanks 👍🏻
If you can, please try to run make fix-all and then make test-all.

I think this is the last issue i could work on easily, other look very domain specific.

I can happily suggest a few easy-ish issues to work on, if you are open to that. That would be much appreciated. There are plenty in different categories.

@rupakbajgain
Copy link
Contributor Author

Ok i removed those errors,
Feel free to tag me on issues—I’ll take a look when I get notified.

@Alkarex
Copy link
Member

Alkarex commented Jan 23, 2026

I have made a few adjustments. Please double-check

@Alkarex Alkarex added the UX User experience label Jan 23, 2026
@rupakbajgain
Copy link
Contributor Author

@Alkarex i was getting warning for those long lines, make fix-all and make test-all were failing without it on mine.
Revert unrelated changes.
image

I manually tested, it looks ok.

@Alkarex Alkarex merged commit 85f14f9 into FreshRSS:edge Jan 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UX User experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Message for closed registration

2 participants