sts-sponsors team mailing list archive
-
sts-sponsors team
-
Mailing list archive
-
Message #06389
Re: [Merge] ~thorsten-merten/maas-site-manager:MAASENG-1497-pagination into maas-site-manager:main
Review: Needs Fixing
Diff comments:
> diff --git a/backend/msm/__init__.py b/backend/msm/__init__.py
> index 8506985..3651249 100644
> --- a/backend/msm/__init__.py
> +++ b/backend/msm/__init__.py
> @@ -1,8 +1,16 @@
> from pkg_resources import get_distribution
>
> -__all__ = ["PACKAGE", "__version__"]
> +__all__ = [
> + "PACKAGE",
> + "__version__",
> + "__default_page_size__",
> + "__max_page_size__",
> +]
>
>
> PACKAGE = get_distribution("msm")
>
> __version__ = PACKAGE.version
> +
> +__default_page_size__ = 20
these shouldn't be dunder variables
> +__max_page_size__ = 100
--
https://code.launchpad.net/~thorsten-merten/maas-site-manager/+git/maas-site-manager/+merge/439629
Your team MAAS Committers is subscribed to branch ~thorsten-merten/maas-site-manager:MAASENG-1497-pagination.
Follow ups
References