Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Social sqlalchemy #1726

Closed

Conversation

NyuydineBill
Copy link
Contributor

@NyuydineBill NyuydineBill commented Mar 8, 2024

Description:
This pull request adds optional instructions for configuring social_sqlalchemy, a package for SQLAlchemy storage support for Python Social Auth, in Anitya . It provides a code example demonstrating how to set up social_sqlalchemy in a Anitya , including configuring the SQLAlchemy database and initializing social_sqlalchemy. Additionally, it includes a note on starting the development web server with Flask.

Purpose:
The purpose of this pull request is to enhance the contributing guidelines by providing guidance on integrating social_sqlalchemy with Anitya for OAuth and OAuth2 authentication. This addition aims to assist contributors who may want to use social_sqlalchemy in Anitya project, thereby improving the overall development experience for Anitya contributors.

Changes Made:
Added instructions for configuring social_sqlalchemy in Flask applications to the contributing.rst file.
Included a code example demonstrating the setup process for social_sqlalchemy.
Updated the Python Virtualenv section of the contributing.rst file to include these instructions.
Testing:
These instructions were tested locally by following the provided steps to configure social_sqlalchemy in a Anitya application within a Python virtual environment. The Flask development server was started successfully, indicating that the setup process was correct and functional.

Related Issue:
#1723

…or resolving server errors during login

Added New file Auth_ERROR-README.md with the description on how to fix this error
…or resolving server errors during login

Added New file Auth_ERROR-README.md with the description on how to fix this error
This commit adds optional instructions for configuring social_sqlalchemy, a package for SQLAlchemy storage support for Python Social Auth, in Flask applications. It provides a code example demonstrating how to set up social_sqlalchemy in a Flask app, including configuring the SQLAlchemy database and initializing social_sqlalchemy. Additionally, it includes a note on starting the development web server with Flask.

These instructions are added to the contributing.rst file under the Python Virtualenv section to provide guidance to contributors who may want to use social_sqlalchemy for OAuth and OAuth2 authentication in their Flask applications.

Resolves: #issue_number:fedora-infra#1723
This commit adds optional instructions for configuring social_sqlalchemy, a package for SQLAlchemy storage support for Python Social Auth, in Flask applications. It provides a code example demonstrating how to set up social_sqlalchemy in a Flask app, including configuring the SQLAlchemy database and initializing social_sqlalchemy. Additionally, it includes a note on starting the development web server with Flask.

These instructions are added to the contributing.rst file under the Python Virtualenv section to provide guidance to contributors who may want to use social_sqlalchemy for OAuth and OAuth2 authentication in their Flask applications.

Resolves: #issue_number:fedora-infra#1723
@NyuydineBill NyuydineBill requested a review from a team as a code owner March 8, 2024 10:36
Copy link

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/5c271a494aec4be28246e0c29d2cc5f2

✔️ fi-tox-mypy SUCCESS in 7m 25s
✔️ fi-tox-lint SUCCESS in 7m 21s
✔️ fi-tox-format SUCCESS in 7m 24s
✔️ fi-tox-python38 SUCCESS in 11m 59s
✔️ fi-tox-python39 SUCCESS in 11m 36s
✔️ fi-tox-python310 SUCCESS in 11m 49s
✔️ fi-tox-python311 SUCCESS in 11m 15s
✔️ fi-tox-docs SUCCESS in 8m 52s
✔️ fi-tox-bandit SUCCESS in 7m 34s
✔️ fi-tox-diff-cover SUCCESS in 12m 18s

docs/contributing.rst Outdated Show resolved Hide resolved
@Zlopez
Copy link
Contributor

Zlopez commented Mar 8, 2024

And also don't forget to add news file for the update. See https://anitya.readthedocs.io/en/latest/contributing.html#release-notes for more info

Copy link

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/5dbd1d8da0f5497286cd2b7c0dc1578d

✔️ fi-tox-mypy SUCCESS in 6m 17s
✔️ fi-tox-lint SUCCESS in 5m 33s
✔️ fi-tox-format SUCCESS in 5m 39s
✔️ fi-tox-python38 SUCCESS in 10m 01s
✔️ fi-tox-python39 SUCCESS in 9m 43s
✔️ fi-tox-python310 SUCCESS in 9m 17s
✔️ fi-tox-python311 SUCCESS in 10m 38s
✔️ fi-tox-docs SUCCESS in 6m 36s
✔️ fi-tox-bandit SUCCESS in 5m 37s
✔️ fi-tox-diff-cover SUCCESS in 11m 03s

@NyuydineBill
Copy link
Contributor Author

And also don't forget to add news file for the update. See https://anitya.readthedocs.io/en/latest/contributing.html#release-notes for more info

Done Sir

Copy link

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/f3f42b64cd004d908d180a47466f14f1

✔️ fi-tox-mypy SUCCESS in 11m 48s
✔️ fi-tox-lint SUCCESS in 6m 32s
✔️ fi-tox-format SUCCESS in 6m 24s
✔️ fi-tox-python38 SUCCESS in 15m 44s
✔️ fi-tox-python39 SUCCESS in 9m 42s
✔️ fi-tox-python310 SUCCESS in 16m 43s
✔️ fi-tox-python311 SUCCESS in 17m 08s
✔️ fi-tox-docs SUCCESS in 13m 53s
✔️ fi-tox-bandit SUCCESS in 11m 51s
✔️ fi-tox-diff-cover SUCCESS in 18m 22s

Copy link
Contributor

@Zlopez Zlopez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still some changes required, but getting there.

docs/contributing.rst Outdated Show resolved Hide resolved
news/1723.other Outdated Show resolved Hide resolved
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/0e7d56c2da864c099a0aeb88d3e2f7f9

✔️ fi-tox-mypy SUCCESS in 9m 18s
✔️ fi-tox-lint SUCCESS in 8m 15s
✔️ fi-tox-format SUCCESS in 8m 56s
✔️ fi-tox-python38 SUCCESS in 12m 47s
✔️ fi-tox-python39 SUCCESS in 12m 45s
✔️ fi-tox-python310 SUCCESS in 12m 10s
✔️ fi-tox-python311 SUCCESS in 10m 11s
fi-tox-docs FAILURE in 9m 44s
✔️ fi-tox-bandit SUCCESS in 6m 19s
✔️ fi-tox-diff-cover SUCCESS in 8m 42s

Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/7233c96f5a374238b6769979d78319c9

✔️ fi-tox-mypy SUCCESS in 5m 49s
✔️ fi-tox-lint SUCCESS in 7m 13s
✔️ fi-tox-format SUCCESS in 5m 37s
✔️ fi-tox-python38 SUCCESS in 10m 01s
✔️ fi-tox-python39 SUCCESS in 10m 44s
✔️ fi-tox-python310 SUCCESS in 9m 12s
✔️ fi-tox-python311 SUCCESS in 8m 50s
fi-tox-docs FAILURE in 7m 14s
✔️ fi-tox-bandit SUCCESS in 6m 37s
✔️ fi-tox-diff-cover SUCCESS in 9m 03s

Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/fd48b0b384b04add9bee67668ae5c89c

✔️ fi-tox-mypy SUCCESS in 7m 29s
✔️ fi-tox-lint SUCCESS in 6m 46s
✔️ fi-tox-format SUCCESS in 6m 02s
✔️ fi-tox-python38 SUCCESS in 10m 48s
✔️ fi-tox-python39 SUCCESS in 10m 40s
✔️ fi-tox-python310 SUCCESS in 10m 37s
✔️ fi-tox-python311 SUCCESS in 10m 40s
fi-tox-docs FAILURE in 8m 30s
✔️ fi-tox-bandit SUCCESS in 6m 07s
fi-tox-diff-cover FAILURE in 8m 36s

@Zlopez Zlopez mentioned this pull request Mar 11, 2024
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/326dd73cf47b42489ecbd6474925b516

✔️ fi-tox-mypy SUCCESS in 7m 41s
✔️ fi-tox-lint SUCCESS in 5m 42s
✔️ fi-tox-format SUCCESS in 4m 49s
✔️ fi-tox-python38 SUCCESS in 11m 08s
✔️ fi-tox-python39 SUCCESS in 10m 50s
✔️ fi-tox-python310 SUCCESS in 9m 00s
✔️ fi-tox-python311 SUCCESS in 10m 41s
fi-tox-docs FAILURE in 8m 25s
✔️ fi-tox-bandit SUCCESS in 4m 59s
✔️ fi-tox-diff-cover SUCCESS in 11m 25s

Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/c52d0e0fe82e4ac1b8accf9165ccbf8b

✔️ fi-tox-mypy SUCCESS in 6m 43s
✔️ fi-tox-lint SUCCESS in 5m 51s
✔️ fi-tox-format SUCCESS in 5m 52s
✔️ fi-tox-python38 SUCCESS in 11m 12s
✔️ fi-tox-python39 SUCCESS in 11m 39s
✔️ fi-tox-python310 SUCCESS in 10m 02s
✔️ fi-tox-python311 SUCCESS in 11m 39s
fi-tox-docs FAILURE in 7m 21s
✔️ fi-tox-bandit SUCCESS in 7m 01s
✔️ fi-tox-diff-cover SUCCESS in 10m 59s

Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/bc18b1c26c9c4b448368e66e7ff24823

✔️ fi-tox-mypy SUCCESS in 8m 36s
✔️ fi-tox-lint SUCCESS in 5m 17s
✔️ fi-tox-format SUCCESS in 6m 47s
✔️ fi-tox-python38 SUCCESS in 9m 33s
✔️ fi-tox-python39 SUCCESS in 12m 04s
✔️ fi-tox-python310 SUCCESS in 9m 39s
✔️ fi-tox-python311 SUCCESS in 11m 50s
fi-tox-docs FAILURE in 8m 28s
✔️ fi-tox-bandit SUCCESS in 5m 35s
✔️ fi-tox-diff-cover SUCCESS in 12m 37s

Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/849b766a51e3422f93775bd2247e6b82

✔️ fi-tox-mypy SUCCESS in 6m 04s
✔️ fi-tox-lint SUCCESS in 6m 35s
✔️ fi-tox-format SUCCESS in 5m 06s
✔️ fi-tox-python38 SUCCESS in 9m 07s
✔️ fi-tox-python39 SUCCESS in 10m 11s
✔️ fi-tox-python310 SUCCESS in 10m 21s
✔️ fi-tox-python311 SUCCESS in 9m 09s
fi-tox-docs FAILURE in 8m 05s
✔️ fi-tox-bandit SUCCESS in 6m 38s
✔️ fi-tox-diff-cover SUCCESS in 10m 43s

Copy link

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/231046ffc89b4a8f8f1a786711376414

✔️ fi-tox-mypy SUCCESS in 7m 19s
✔️ fi-tox-lint SUCCESS in 4m 33s
✔️ fi-tox-format SUCCESS in 4m 59s
✔️ fi-tox-python38 SUCCESS in 10m 33s
✔️ fi-tox-python39 SUCCESS in 10m 49s
✔️ fi-tox-python310 SUCCESS in 9m 20s
✔️ fi-tox-python311 SUCCESS in 8m 49s
✔️ fi-tox-docs SUCCESS in 8m 05s
✔️ fi-tox-bandit SUCCESS in 5m 05s
✔️ fi-tox-diff-cover SUCCESS in 9m 49s

Copy link

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/c75d134727084f46aa39a42ff64b6439

✔️ fi-tox-mypy SUCCESS in 5m 49s
✔️ fi-tox-lint SUCCESS in 5m 11s
✔️ fi-tox-format SUCCESS in 5m 40s
✔️ fi-tox-python38 SUCCESS in 9m 44s
✔️ fi-tox-python39 SUCCESS in 10m 53s
✔️ fi-tox-python310 SUCCESS in 13m 02s
✔️ fi-tox-python311 SUCCESS in 10m 47s
✔️ fi-tox-docs SUCCESS in 9m 58s
✔️ fi-tox-bandit SUCCESS in 6m 41s
✔️ fi-tox-diff-cover SUCCESS in 13m 36s

@Zlopez
Copy link
Contributor

Zlopez commented Mar 11, 2024

Thanks for fixing the tests! Could you now squash the commits so there is only one commit message?

Copy link

Build succeeded.
https://fedora.softwarefactory-project.io/zuul/buildset/f43c1638c6ca457e8db0692e5cc86b8a

✔️ fi-tox-mypy SUCCESS in 7m 07s
✔️ fi-tox-lint SUCCESS in 4m 49s
✔️ fi-tox-format SUCCESS in 6m 30s
✔️ fi-tox-python38 SUCCESS in 9m 18s
✔️ fi-tox-python39 SUCCESS in 8m 51s
✔️ fi-tox-python310 SUCCESS in 10m 18s
✔️ fi-tox-python311 SUCCESS in 10m 09s
✔️ fi-tox-docs SUCCESS in 6m 36s
✔️ fi-tox-bandit SUCCESS in 4m 52s
✔️ fi-tox-diff-cover SUCCESS in 9m 32s

Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci
https://fedora.softwarefactory-project.io/zuul/buildset/e158a3c044504e24b51f6180d45070fa

✔️ fi-tox-mypy SUCCESS in 7m 26s
✔️ fi-tox-lint SUCCESS in 5m 16s
✔️ fi-tox-format SUCCESS in 6m 42s
fi-tox-python38 FAILURE in 8m 08s
✔️ fi-tox-python39 SUCCESS in 11m 03s
✔️ fi-tox-python310 SUCCESS in 9m 02s
✔️ fi-tox-python311 SUCCESS in 11m 16s
✔️ fi-tox-docs SUCCESS in 8m 09s
✔️ fi-tox-bandit SUCCESS in 5m 19s
✔️ fi-tox-diff-cover SUCCESS in 11m 39s

@NyuydineBill NyuydineBill deleted the social_sqlalchemy branch March 12, 2024 14:01
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.

2 participants