Fix: url encode workflows external id

[ddonukis]

Description

Workflows service allows arbitrary strings as workflow external id or version id. However, these are used as url path parameters and should be url-encoded. Fixing it now.

Checklist:

• Tests added/updated.
• Documentation updated. Documentation is generated from docstrings - these must be updated according to your change.
  If a new method has been added it should be referenced in cognite.rst in order to generate docs based on its docstring.
• Changelog updated in CHANGELOG.md.
• Version bumped. If triggering a new release is desired, bump the version number in _version.py and pyproject.toml per semantic versioning.

[codecov]

Codecov Report

Merging #1519 (d6c5354) into master (4912644) will increase coverage by 0.07%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1519      +/-   ##
==========================================
+ Coverage   91.72%   91.80%   +0.07%     
==========================================
  Files         120      120              
  Lines       15587    15588       +1     
==========================================
+ Hits        14297    14310      +13     
+ Misses       1290     1278      -12     

Files	Coverage Δ
cognite/client/_api/workflows.py	81.35% <100.00%> (+0.15%)	⬆️
cognite/client/_version.py	100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

[doctrino]

There are a few more strings used directly in the URL in the workflows.py, 103, 139, and 206.

In addition, is this worth a simple test?

[ddonukis]

There are a few more strings used directly in the URL in the workflows.py, 103, 139, and 206.

In addition, is this worth a simple test?

The other url path params are provided by the Workflows service and are uuids, that's why I didn't update those.

Do you think they should be quoted anyways to be safe?

[doctrino]

There are a few more strings used directly in the URL in the workflows.py, 103, 139, and 206.
In addition, is this worth a simple test?

The other url path params are provided by the Workflows service and are uuids, that's why I didn't update those.

Do you think they should be quoted anyways to be safe?

Ah, no I don't think it is necessary.

[haakonvt]

In addition, is this worth a simple test?

@ddonukis 👀