Use get_paginated_data when retrieving list of domains

[elcfd]

SUMMARY

Fixes #306 by using get_paginated_data

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

digital_ocean_domain module

ADDITIONAL INFORMATION

[elcfd]

Hi @mamercad apologies for bothering you like this but is there an ETA on when this can be looked at? Currently having to run this module with a patch in my infrastructure. Thanks @elcfd

[mamercad]

Hi @mamercad apologies for bothering you like this but is there an ETA on when this can be looked at? Currently having to run this module with a patch in my infrastructure. Thanks @elcfd

Apologies, I've been swamped at $work, looking now.

[mamercad]

Thanks for this, please kindly add a changelog fragment.

[elcfd]

Hi @mamercad no problem - have added the changelog fragement as requested.

[codecov]

Codecov Report

Merging #307 (6d16808) into main (0946170) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #307   +/-   ##
=======================================
  Coverage   31.09%   31.10%           
=======================================
  Files          45       45           
  Lines        3589     3588    -1     
  Branches      732      732           
=======================================
  Hits         1116     1116           
+ Misses       2453     2452    -1     
  Partials       20       20           

Flag	Coverage Δ
sanity	26.42% <0.00%> (+<0.01%)	⬆️
units	64.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
plugins/modules/digital_ocean_domain.py	20.89% <0.00%> (+0.15%)	⬆️

[mamercad]

I think we'll need #310 before this passes tests, I'm working on it.

[mamercad]

I think we'll need #310 before this passes tests, I'm working on it.

@elcfd kindly rebase this against main and let's see how the tests look.

[elcfd]

@mamercad have rebased against main as requested

[mamercad]

Thanks for this ❤️

[elcfd]

Thanks for this heart

@mamercad not a problem - glad to have sorted the bug - thanks for the help merging