Skip to content

Conversation

@datamweb
Copy link
Contributor

@datamweb datamweb commented Jan 2, 2026

Description

CodeIgniter4 behaves significantly differently depending on the environment.

Checklist:

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

attributes:
label: Environment
description: Which CI_ENVIRONMENT setting are you using?
options:
Copy link
Member

Choose a reason for hiding this comment

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

This should be multi-select options as the bug can occur in multiple envs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

Copy link
Collaborator

@ddevsr ddevsr Jan 2, 2026

Choose a reason for hiding this comment

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

when self-env? i think must have it

Copy link
Member

Choose a reason for hiding this comment

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

"self-env" in itself seems vague. is there a way to write it like "others, please specify" with a text box for that others?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems that GitHub currently do not support conditional logic (showing a field dependent on a specific choice).

https://github.com/orgs/community/discussions/4386

To implement this, we would have to add a separate input field that is always visible (though optional).

I'm not personally fond of this approach as it clutters the form for users who don't need it. Given this limitation, would you still like me to add it?

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