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

Register News block (#30) #32

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Register News block (#30) #32

wants to merge 1 commit into from

Conversation

MlKilderkin
Copy link

@MlKilderkin MlKilderkin commented Nov 14, 2024

What does this do/fix?

  • Register News block
  • update scripts and adds base block files and config updates.
  • Add assets build for the plugin
  • upgrade wpscripts to the latest version.
  • adds linting support and npm scripts updates for wpscripts.
  • update block instructions.
  • update dev watch task.
  • Update assets enqueue
  • adds nvmrc file.
  • re-add news link items.
  • adds more news button styles.
  • adds layout setting styles.
  • fix button styles.
  • limit terms to 3 each.
  • fix empty array issue.
  • Add preview text for empty block in admin

* Register News block
* update scripts and adds base block files and config updates.
* Add assets build for the plugin
* upgrade wpscripts to the latest version.
* adds linting support and npm scripts updates for wpscripts.
* update block instructions.
* update dev watch task.
* Update assets enqueue
* adds nvmrc file.
* re-add news link items.
* adds more news button styles.
* adds layout setting styles.
* fix button styles.
* limit terms to 3 each.
* fix empty array issue.
* Add preview text for empty block in admin

---------

Co-authored-by: Vinny Listrani <[email protected]>
@MlKilderkin
Copy link
Author

Sorry @knice It looks like we point #30 to our sprint branch but didn't point it to the correct place. Would you please take a look.

@knice
Copy link
Member

knice commented Nov 14, 2024

@MlKilderkin Should I review and merge this one? Or do I need to do something else?

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.

2 participants