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

feature/Ny modul Modal CU 865cn09xb #441

Merged
merged 12 commits into from
Aug 7, 2023

Conversation

annalinneajohansson
Copy link
Contributor

  • (add) ACF fields for module Modal
  • (add) New module "Modal"

@helsingborgsstad
Copy link

Task linked: CU-865cn09xb Ny modul: Modal

@annalinneajohansson annalinneajohansson marked this pull request as ready for review July 5, 2023 08:07
Copy link
Member

@sebastianthulin sebastianthulin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small changes, otherwise great!

* @param object $editorContext The editor context.
* @return array The updated allowed block types.
*/
public function disallowBlockType($allowedBlockTypes, $editorContext)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

@annalinneajohansson annalinneajohansson Jul 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not following your thought here; can you please explain? The module is meant to be available as a block, but not on the modal-content post type. I'm not finding any way to detect the post type currently being edited whilst registering the module.

source/php/Module/Modal/Modal.php Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Jul 24, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2128 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sebastianthulin sebastianthulin merged commit 5f511d1 into 3.0/develop Aug 7, 2023
3 of 4 checks passed
@sebastianthulin sebastianthulin deleted the feature/Ny-modul-Modal-CU-865cn09xb branch August 7, 2023 12:56
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