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

Init #1

Merged
merged 105 commits into from
Jan 19, 2024
Merged

Init #1

merged 105 commits into from
Jan 19, 2024

Conversation

iruzevic
Copy link
Member

@iruzevic iruzevic commented Jan 5, 2024

Description

  • extracting funkcionality.

mbmjertan
mbmjertan previously approved these changes Jan 18, 2024
Copy link
Collaborator

@mbmjertan mbmjertan left a comment

Choose a reason for hiding this comment

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

looks okay

@iruzevic iruzevic merged commit d4fabf9 into main Jan 19, 2024
4 checks passed
Copy link

@piqusy piqusy left a comment

Choose a reason for hiding this comment

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

Haven't reviewed everything bc the PR got merged while reviewing 😄

@@ -0,0 +1,3 @@
# Project Specific
/vendor
/.vscode
Copy link

Choose a reason for hiding this comment

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

Suggested change
/.vscode
/.vscode
# Mac OS custom attribute store and thumbnails
*.DS_Store
._*


# Eightshift Forms Utils Library

Eightshift Forms Utils library is a set of helpers and utils that can be used in the main Eightshift Forms plugin or additional Add-on plugins.
Copy link

Choose a reason for hiding this comment

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

Suggested change
Eightshift Forms Utils library is a set of helpers and utils that can be used in the main Eightshift Forms plugin or additional Add-on plugins.
Eightshift Forms Utils library is a set of helpers and utils that can be used in the main Eightshift Forms plugin or additional Add-ons.

use EightshiftLibs\Blocks\AbstractBlocks;

/**
* Class AbstractUtilsBaseRoute
Copy link

Choose a reason for hiding this comment

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

Suggested change
* Class AbstractUtilsBaseRoute
* Class UtilsBlocks

<?php

/**
* Class Blocks is the base class for Gutenberg blocks registration.
Copy link

Choose a reason for hiding this comment

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

Suggested change
* Class Blocks is the base class for Gutenberg blocks registration.
* Class UtilsBlocks is the base class for Gutenberg blocks registration.

Comment on lines +445 to +448
public const API_RESPONSE_CODE_ERROR = 400;
public const API_RESPONSE_CODE_ERROR_MISSING = 404;
public const API_RESPONSE_CODE_ERROR_SERVER = 500;
public const API_RESPONSE_CODE_ERROR_FORBIDDEN = 403;
Copy link

Choose a reason for hiding this comment

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

Suggested change
public const API_RESPONSE_CODE_ERROR = 400;
public const API_RESPONSE_CODE_ERROR_MISSING = 404;
public const API_RESPONSE_CODE_ERROR_SERVER = 500;
public const API_RESPONSE_CODE_ERROR_FORBIDDEN = 403;
public const API_RESPONSE_CODE_ERROR = 400;
public const API_RESPONSE_CODE_ERROR_FORBIDDEN = 403;
public const API_RESPONSE_CODE_ERROR_MISSING = 404;
public const API_RESPONSE_CODE_ERROR_SERVER = 500;

}

/**
* Return counries filtered by some key for multiple usages.
Copy link

Choose a reason for hiding this comment

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

Suggested change
* Return counries filtered by some key for multiple usages.
* Return countries filtered by some key for multiple usages.

}

/**
* Output select options ass array from html string.
Copy link

Choose a reason for hiding this comment

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

Suggested change
* Output select options ass array from html string.
* Output select options as array from HTML string.

Comment on lines +841 to +843
* Clean url from query params.
*
* @param string $url Url to clean.
Copy link

Choose a reason for hiding this comment

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

Suggested change
* Clean url from query params.
*
* @param string $url Url to clean.
* Clean URL from query params.
*
* @param string $url URL to clean.

'layoutContent' => [
[
'component' => 'intro',
'introTitle' => \__('Disclamer', 'eightshift-forms'),
Copy link

Choose a reason for hiding this comment

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

Suggested change
'introTitle' => \__('Disclamer', 'eightshift-forms'),
'introTitle' => \__('Disclaimer', 'eightshift-forms'),

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.

4 participants