Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update ArticleContent.php #20

Closed
wants to merge 2 commits into from
Closed

Update ArticleContent.php #20

wants to merge 2 commits into from

Conversation

arboc
Copy link

@arboc arboc commented Apr 8, 2024 •
edited
Loading

Description

Closes #19

Checklist

  • Read and understood the CONTRIBUTING guidelines
  • Created tests, if possible
  • All tests passing
  • Extended the README / documentation, if necessary
  • Added myself to the @authors in touched PHP files
  • Checked the changes with phpcq and introduced no new issues

@arboc
Copy link
Author

arboc commented Apr 9, 2024

added @author

@zonky2 zonky2 requested a review from stefanheimes April 9, 2024 13:03
@zonky2 zonky2 added the enhancement New feature or request label Apr 9, 2024
@zonky2 zonky2 modified the milestones: 2.2.0, 2.2.2 Apr 9, 2024
@zonky2
Copy link
Contributor

zonky2 commented Apr 9, 2024

Method checkAccessToElement is no longer included in MM 2.3

@zonky2
Copy link
Contributor

zonky2 commented Aug 1, 2024

Attribute have own controller in MM 2.3 - fix is obsolete

@zonky2 zonky2 closed this Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stefanheimes stefanheimes Awaiting requested review from stefanheimes

Assignees

@arboc arboc

Labels
enhancement New feature or request
Projects
None yet
Milestone
2.2.2
Development

Successfully merging this pull request may close these issues.

no access when not admin
2 participants
@arboc @zonky2