Skip to content

Added the options function to polls and quizzes#765

Open
Kawesom wants to merge 5 commits into
defstudio:mainfrom
Kawesom:main
Open

Added the options function to polls and quizzes#765
Kawesom wants to merge 5 commits into
defstudio:mainfrom
Kawesom:main

Conversation

@Kawesom
Copy link
Copy Markdown

@Kawesom Kawesom commented May 12, 2026

  • this allows you to pass an array of options to a poll/quizz as opposed to how it was previously done by calling the option function for each item you wanted to added.
  • this PR includes tests for the new function as well.

- this allows you to pass an array of options to a poll/quizz as opposed to how it was previously done by calling the option function for each item you wanted to added.
- this PR includes tests for the new function as well.
@fabio-ivona fabio-ivona requested a review from MarioGattolla May 13, 2026 15:12
@MarioGattolla
Copy link
Copy Markdown
Contributor

Hi @Kawesom Thanks for the merge request, can you update the documentation with the new method. That would be really appreciated

@Kawesom
Copy link
Copy Markdown
Author

Kawesom commented May 22, 2026

Hi @Kawesom Thanks for the merge request, can you update the documentation with the new method. That would be really appreciated

Ok, will do

- changed error message in tooManyOptions TelegraphpollException to reflect actual limit which is 12 and not 10.
- added documentation for new poll/quiz $options method
- fix minor issue with the $options function for quizzes
@Kawesom
Copy link
Copy Markdown
Author

Kawesom commented May 22, 2026

I've added the documentation as you requested

@Kawesom
Copy link
Copy Markdown
Author

Kawesom commented May 26, 2026

Hi @Kawesom Thanks for the merge request, can you update the documentation with the new method. That would be really appreciated

I keep getting the " WARN No code coverage driver available" from github actions, don't know how to resolve that

@MarioGattolla
Copy link
Copy Markdown
Contributor

@Kawesom in the "run-test.yml" file you need to edit the "execute test" step like this without coverage

  - name: Execute tests
        run: vendor/bin/pest --ci -vv --no-coverage

@Kawesom
Copy link
Copy Markdown
Author

Kawesom commented May 26, 2026

@Kawesom in the "run-test.yml" file you need to edit the "execute test" step like this without coverage


  - name: Execute tests

        run: vendor/bin/pest --ci -vv --no-coverage



Ok, will do

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