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

"Suggest a change on this page" button bug/change request #16654

Open
JavierLopezT opened this issue Jun 25, 2021 · 15 comments
Open

"Suggest a change on this page" button bug/change request #16654

JavierLopezT opened this issue Jun 25, 2021 · 15 comments
Assignees

Comments

@JavierLopezT
Copy link
Contributor

JavierLopezT commented Jun 25, 2021

Coming from google to the following web: https://airflow.apache.org/docs/apache-airflow-providers-amazon/stable/_api/airflow/providers/amazon/aws/hooks/s3/index.html

The button 'Suggest a change on this page' doesn't work, it leads you to https://github.com/apache/airflow/edit/main/docs/apache-airflow-providers-amazon/_api/airflow/providers/amazon/aws/hooks/s3/index.rst
Captura de pantalla 2021-06-25 a las 14 36 56

Also, I think the button is way too big and it makes "difficult" to read the documentation
Captura de pantalla 2021-06-25 a las 14 38 04

@JavierLopezT JavierLopezT added the kind:bug This is a clearly a bug label Jun 25, 2021
@potiuk
Copy link
Member

potiuk commented Jun 25, 2021

Yep. Agree. The API documentation is automatically generated and the "suggest a change" button makes no sense there. Maybe you could try to figure out how to remove it @JavierLopezT and make a PR to fix it ? @mik-laj might provide some hints.

@ShivamJhaa
Copy link

I will like to fix this bug.
Please assign me the task.
Thank you

@potiuk
Copy link
Member

potiuk commented Jun 26, 2021

assigned!

@eladkal
Copy link
Contributor

eladkal commented Feb 16, 2022

@ShivamJhaa are you still working on this issue?

@Megha-Sai-Sree-PINNAKA
Copy link

Hello, is this issue still available? If yes, I would like to try it.

@ShivamJhaa ShivamJhaa removed their assignment Jun 10, 2022
@potiuk
Copy link
Member

potiuk commented Jun 10, 2022

Sure.

@potiuk
Copy link
Member

potiuk commented Jun 10, 2022

Assigned you

@Megha-Sai-Sree-PINNAKA
Copy link

Hello @potiuk. You mentioned that removing the suggest a change button would be better right as the documentation is automatically generated. Do you mean removing it in all the web pages or only from here https://airflow.apache.org/docs/apache-airflow-providers-amazon/stable/_api/airflow/providers/amazon/aws/hooks/s3/index.html?

@potiuk
Copy link
Member

potiuk commented Jun 17, 2022

All the auto-generated pages - those that have no corresponding .rst file but are generated from the python code.

@Megha-Sai-Sree-PINNAKA
Copy link

Hello @potiuk and @mik-laj

I understood that there are 2 ways to display the documentation on airflow website.

  1. Through .rst files where any changes can be directly done on this file and will be reflected on the corresponding UI Page.
  2. Through .py file. In case of amazon aws hooks (s3)the decorators and static methods are being used for the document generation on the HTML Page. But don’t have a default .rst file to make any direct changes so that it gets displayed on the UI.

Can you please provide me any pointers to know how the airflow site pages are being invoked from apache airflow repo?

@potiuk
Copy link
Member

potiuk commented Jun 24, 2022

The .py files are generated by autoapi plugin - that's what I know. once the build is done, they are all generated in _api subfolder, so the most likely way to check it is to use the location of the source. But how exactly to do it, I am not sure - I am not a sphinx expert :)

@mik-laj
Copy link
Member

mik-laj commented Jun 24, 2022

Documentation for class is generated based on python files.

@qleroy
Copy link
Contributor

qleroy commented Aug 11, 2023

The button is still way too large and makes parts of the page unreadable.
Screenshot 2023-08-11 at 09 20 24

@potiuk
Copy link
Member

potiuk commented Aug 11, 2023

The button is still way too large and makes parts of the page unreadable. Screenshot 2023-08-11 at 09 20 24

PRs to decreese it are most welcome. I think it's great to get it easy to contribute back to the documentation and many of our contributor use it. But If you feel it's too large @qleroy - I heartily invite you to do your first contribution and for example make a CSS that will adapt it, depending on the amount of available screen real estate. That would be great contribution.

@shivannakarthik
Copy link

@potiuk - If this is still open, I would like to pick it up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants