Skip to content

1.0 #58

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
Merged
merged 147 commits into from
Mar 23, 2023
Merged

1.0 #58

merged 147 commits into from
Mar 23, 2023

Conversation

@n7studios
Copy link
Contributor

@n7studios n7studios commented Mar 23, 2023

Summary

Contains all approved PRs for the 1.0 release of the PHP SDK.

Testing

Existing tests pass.

Checklist

n7studios added 30 commits March 1, 2023 16:22
@n7studios
Bump minimum versions for dependencies; require PHP 7.4 or higher
37abed0
@n7studios
Remove trailing comma
1bd3188
@n7studios
Updated .gitignore file to exclude vendor folder and other local files
43a7f07
@n7studios
Started work on testing and pull request guides
1253f9d
@n7studios
Ignore .phpunit.result.cache
6a6a078
@n7studios
Fix incorrect PHPUnit configuration
b9cf8a7
@n7studios
Make tests compatible with PHPUnit 9.x
d092c79
@n7studios
Use .env file for configuration, replacing tests/config.example.php
7df4587
@n7studios
Merge pull request #28 from ConvertKit/fix-dependencies
ce53870 
Bump dependencies
@n7studios
Merge remote-tracking branch 'origin/1.0-beta' into fix-tests
556798a
@n7studios
Started work on refactoring and improving test coverage
16f51f9
@n7studios
More work on improving test coverage
b678865
@n7studios
Completed first pass of improved test coverage
306b6ec
@n7studios
Updated .env files
134772f
@n7studios
Started applying coding standards to tests
ed0ebd9
@n7studios
Just use PSR-12 for coding standards for now
fa34ce8
@n7studios
First pass at GitHub action to automate tests
d77b521
@n7studios
Update testing guide
878634d
@n7studios
Query subscibers by email address, instead of using inefficient pagin…
9ce7380 
…g method
@n7studios
Implement code from ConvertKit API library for fetching legacy forms …
ef52956 
…and legacy landing pages
@n7studios
Remove unused depdencies
f9445fd
@n7studios
Remove unused .env variables
0b7f052
@n7studios
Improved test coverage for get_resource()
2317d4c
@n7studios
Add missing CONVERTKIT_API_SEQUENCE_ID in .env files
eab7d04
@n7studios
Apply PSR-12 coding standards to API class
b4e79cc
@n7studios
Remove blank lines after opening function brace
e197ec8
@n7studios
Updated coding standard rules to favor PSR-12, and import/honor Squiz…
9dc57ff 
… docblock standards
@n7studios
Fix TypeError on type hinting DOMNodeList
2664982
@n7studios
Merge pull request #33 from ConvertKit/coding-standards
4cda118 
Apply Coding Standards
@n7studios
Merge pull request #32 from ConvertKit/fix-get-resource
366a9e7 
Improve `get_resource()`
n7studios added 26 commits March 21, 2023 15:56
@n7studios
Updated readme to link to docs generated by phpDocumentor
ae02679
@n7studios
Updated guides for deployment, setup and contributing
3b0a076
@n7studios
Fix doc links, added screenshots
02875e3
@n7studios
Add releases screenshot; remove WordPress references
5703067
@n7studios
Remove InvalidArgumentException
4747c89
@n7studios
Merge pull request #54 from ConvertKit/add-subscriber-to-form
dd2c992 
Add `add_subscriber_to_form()` function with named arguments
@n7studios
Merge pull request #56 from ConvertKit/1.0-docs
ba74ace 
Add Guides and Documentation
@n7studios
Merge remote-tracking branch 'origin/1.0-beta' into phpdocumentor
58a37eb
@n7studios
Remove saggre/phpdocumentor-markdown in favor of own phpDocumentor ma…
db3bb31 
…rkdown templates to fix encoding issues
@n7studios
Ignore phpDocumentor ast.dump file for debugging
8d786e7
@n7studios
Parameters table: Description: Replace pipe symbol with comma, instea…
4c696d6 
…d of HTML encoded pipe
@n7studios
Updated instructions for compiling docs
974c439
@n7studios
Fix encoding of single quotation marks in method defaults
8e48007
@n7studios
Experiment with removing macros, header and footer templates
e0ba571
@n7studios
Remove unnecessary spacing in markdown docs
10c9cd3
@n7studios
Further improvements to markdown templates for PHP SDK docs
2236811
@n7studios
Remove unused macros
7a5bba1
@n7studios
Fix encoding on method return types
ff9019f
@n7studios
Add newlines between Parameters and Return Value headings
1fdae1c
@n7studios
Escape pipes in tables for better rendering
37c4624
@n7studios
Escape pipes in tables for argument types
c370af0
@n7studios
Double escape pipe symbols in tables to ensure output renders correctly
0ebf834
@n7studios
Merge pull request #55 from ConvertKit/phpdocumentor
dcf9cd1 
Add phpDocumentor generated PHP SDK Documentation
@n7studios
Add missing see tags to functions, linking to API docs
cb17ad5
@n7studios
Merge pull request #57 from ConvertKit/add-see-tags
8c196fb 
Add missing see tags to methods
@n7studios
Merge branch 'master' into 1.0-beta
ac00175
@n7studios n7studios self-assigned this Mar 23, 2023
@n7studios n7studios marked this pull request as ready for review March 23, 2023 15:58
@n7studios n7studios merged commit 2d317ea into master Mar 23, 2023
@n7studios n7studios deleted the 1.0-beta branch March 23, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@n7studios n7studios

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@n7studios