search

globus / globus-sdk-python

Globus SDK for Python
Apache License 2.0
68 stars 38 forks source link

Move globus app #1076

Closed derek-globus closed 1 month ago

derek-globus commented 1 month ago

📚 Documentation preview 📚: https://globus-sdk-python--1076.org.readthedocs.build/en/1076/

derek-globus commented 1 month ago

^ Rebased against main to resolve merge conflicts ahead of review

sirosen commented 1 month ago

I think that given that the new docs link to Using a GlobusApp, now would be a good time to move that doc in particular out of the Updated Examples section. We can punt this to a follow-up PR if you prefer. Here's what the sidebar looks like right now: image

I think Using a GlobusApp could go under Getting Started, alongside the tutorial. Is that okay for an initial step? Docs can always be moved more later.

derek-globus commented 1 month ago

I think that given that the new docs link to Using a GlobusApp, now would be a good time to move that doc in particular out of the Updated Examples section.

I agree but want to punt. This PR is already pretty massive as-is; so unless something is actively broken I think we separate that work.