Skip to content

Conversation

@JLTRY
Copy link
Collaborator

@JLTRY JLTRY commented Nov 21, 2024

I added filter to add only attachments from current article

@parapente
Copy link
Collaborator

Good job @JLTRY! It needs a couple of fixes but seems to work well combined with the other pull request. I'll add the changes in the review as I don't want to mix the two pull requests and push anything wrong to your branch.

@JLTRY
Copy link
Collaborator Author

JLTRY commented Nov 23, 2024

Hello fill free to modify if you have time I am in vacation.I made a dedicated branch for this pr

@parapente
Copy link
Collaborator

@JLTRY in commit 65f6b47 I added two new strings for you to translate to French. I believe that the plugin is ready for use. You can test and then merge to master.

@JLTRY JLTRY merged commit 350109f into master Nov 26, 2024
@JLTRY
Copy link
Collaborator Author

JLTRY commented Nov 26, 2024

@parapente merged. Let me know which version we should set for next release

@parapente
Copy link
Collaborator

@JLTRY I think we should wait to finish #5 before preparing a 4.1.0 version. I want to test a couple of things for this PR too to make sure it works correctly everywhere.

@JLTRY JLTRY deleted the attachmentsid branch December 29, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants