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.

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

fix: permission to avoid any conflict with laravel policy #62

Merged
merged 2 commits into from
Nov 22, 2022

Conversation

matthv
Copy link
Member

@matthv matthv commented Nov 21, 2022

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)

Security

  • Consider the security impact of the changes made

@arnaudbesnier
Copy link
Member

@codeclimate
Copy link

codeclimate bot commented Nov 21, 2022

Code Climate has analyzed commit 0d76e51 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (99% is the threshold).

This pull request will bring the total coverage in the repository to 100.0%.

View more on Code Climate.

@matthv matthv changed the title Fix/forest policy conflict fix: permission to avoid any conflict with laravel policy Nov 22, 2022
@matthv matthv merged commit 03d5856 into main Nov 22, 2022
@matthv matthv deleted the fix/forest-policy-conflict branch November 22, 2022 16:44
forest-bot added a commit that referenced this pull request Nov 22, 2022
## [1.2.2](v1.2.1...v1.2.2) (2022-11-22)

### Bug Fixes

* permission to avoid any conflict with laravel policy ([#62](#62)) ([03d5856](03d5856))
@forest-bot
Copy link
Member

🎉 This PR is included in version 1.2.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

arnaud-moncel pushed a commit that referenced this pull request Jul 17, 2023
## [1.2.2](v1.2.1...v1.2.2) (2022-11-22)

### Bug Fixes

* permission to avoid any conflict with laravel policy ([#62](#62)) ([03d5856](03d5856))
forest-bot added a commit that referenced this pull request Jul 17, 2023
# [1.0.0-beta.24](v1.0.0-beta.23...v1.0.0-beta.24) (2023-07-17)

### Bug Fixes

* **charts:** user with permissions level that allows charts creation or edition should always be allow to perform charts requests ([#60](#60)) ([e33a601](e33a601))
* **query-builder:** patch eagerload relationships on collection ([#66](#66)) ([0c1ef79](0c1ef79))
* **relationship:** ignore methods with parameters that return relationships  ([#68](#68)) ([99d380a](99d380a))
* **security:** update guzzle version (with dependency) to 7.4.5 ([#58](#58)) ([3b26ebd](3b26ebd))
* **security:** update guzzle version to 7.4.3 ([#52](#52)) ([b32b9db](b32b9db))
* **smart-collection:** add support of smart-relationship ([#63](#63)) ([7b69434](7b69434))
* add database type timestamp for schema generation ([#57](#57)) ([087eaea](087eaea))
* permission to avoid any conflict with laravel policy ([#62](#62)) ([883129a](883129a))
* render jsonapi when a model uses eager load relation ([#61](#61)) ([656500c](656500c))

### Features

* add support for multiple models directories ([#56](#56)) ([bde2f5d](bde2f5d))
* allow to use agent with laravel 10 ([#69](#69)) ([69b01cc](69b01cc))
* allow user to onboard to the new eloquent datasource from forestadmin/agent-php ([#72](#72)) ([09f4822](09f4822))
* **auth:** remove callbackUrl parameter on authentication ([#59](#59)) ([647536e](647536e))

### BREAKING CHANGES

* allow user to onboard to the new eloquent datasource from forestadmin/agent-php
arnaud-moncel pushed a commit that referenced this pull request Jul 17, 2023
## [1.2.2](v1.2.1...v1.2.2) (2022-11-22)

### Bug Fixes

* permission to avoid any conflict with laravel policy ([#62](#62)) ([03d5856](03d5856))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants