Skip to content

Conversation

@michalsn
Copy link
Member

Description
This PR adds CNAME and updates the docs URL in readme.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide
## Docs

Read the full documentation: https://codeigniter4.github.io/settings/
Read the full documentation: https://settings.codeigniter.com
Copy link
Contributor

Choose a reason for hiding this comment

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

If you are going to use https://settings.codeigniter.com, it is better to use it in all cases. including file composer.js & mkdocs.yml and...

Copy link
Member Author

Choose a reason for hiding this comment

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

IMO composer.json should link to the source code only.
As for MkDocs... do you mean instead of linking to codeigniter.com in extra.homepage?

Copy link
Contributor

@datamweb datamweb Dec 19, 2023

Choose a reason for hiding this comment

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

IMO composer.json should link to the source code only.

Thanks, I didn't know that.

As for MkDocs... do you mean instead of linking to codeigniter.com in extra.homepage?

Sorry. No, it's fine now, I thought there was a link to the docs defined in MkDocs.

@michalsn
Copy link
Member Author

Thanks all.

@michalsn michalsn merged commit 0d848cf into codeigniter4:develop Dec 19, 2023
@michalsn michalsn deleted the add-cname branch December 22, 2023 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants