Fixed Edit on github links for v13.x documentation

[javad94]

Checklist for PRs

• Added .. versionadded:: version, .. versionchanged:: version or .. deprecated:: version to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)
• Created new or adapted existing unit tests
• Documented code changes according to the CSI standard
• Added myself alphabetically to AUTHORS.rst (optional)
• Added new classes & modules to the docs and all suitable __all__ s

[javad94]

btw jinja2 gives ImportError when using sphinx-build docs/source docs/build/html command. Downgrading jinja2 from 3.1.2 to 3.0.3 fixed this issue. So should I update requirements-docs.txt too?

Traceback (most recent call last):
  File "C:\Users\Javad\Anaconda3\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\Javad\Anaconda3\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "D:\python-projects\ptb\venv\Scripts\sphinx-build.exe\__main__.py", line 4, in <module>
  File "D:\python-projects\ptb\venv\lib\site-packages\sphinx\cmd\build.py", line 25, in <module>
    from sphinx.application import Sphinx
  File "D:\python-projects\ptb\venv\lib\site-packages\sphinx\application.py", line 43, in <module>
    from sphinx.registry import SphinxComponentRegistry
  File "D:\python-projects\ptb\venv\lib\site-packages\sphinx\registry.py", line 24, in <module>
    from sphinx.builders import Builder
  File "D:\python-projects\ptb\venv\lib\site-packages\sphinx\builders\__init__.py", line 26, in <module>
    from sphinx.util import import_object, logging, progress_message, rst, status_iterator
  File "D:\python-projects\ptb\venv\lib\site-packages\sphinx\util\rst.py", line 21, in <module>
    from jinja2 import Environment, environmentfilter
ImportError: cannot import name 'environmentfilter' from 'jinja2' (D:\python-projects\ptb\venv\lib\site-packages\jinja2\__init__.py)

[harshil21]

Thanks for the fix. Is your sphinx version the one specified in requirements-docs.txt? The error probably happens since the versions aren't exactly matching the ones we specified.

[javad94]

Thanks for the fix. Is your sphinx version the one specified in requirements-docs.txt? The error probably happens since the versions aren't exactly matching the ones we specified.

Yes, it is the same version.

[harshil21]

okay since I see you're on a venv you probably have the right versions I'd say to go ahead and add that in (Jinja2 < 3.1). Makes sense you're getting this since I see this in the sphinx source code.

[javad94]

okay since I see you're on a venv you probably have the right versions I'd say to go ahead and add that in (Jinja2 < 3.1). Makes sense you're getting this since I see this in the sphinx source code.

Of course, done.

[javad94]

@harshil21 Anything else needed to merge this PR?

[harshil21]

Thanks for the contribution!

[javad94]

Thanks for the contribution!

You're very welcome.