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

Use external markdown for the spec reference #66

Merged
merged 3 commits into from
Nov 15, 2023

Conversation

richfab
Copy link
Contributor

@richfab richfab commented Nov 15, 2023

Problem

Every time we make a change to the spec in the gbfs.md file, we also need to update the reference on gbfs.org to reflect the change. This is a manual task that takes time and that is prone to human error.

Solution

This PR proposes to use the plugin mkdocs-embed-external-markdown to replace the static page by a dynamic import of the content from gbfs.md from the gbfs repository.

With this solution, every time a change is made in the gbfs.md file, it is automatically reflected on gbfs.org (no action needed).

Before After
Screenshot 2023-11-15 at 14 29 10 Screenshot 2023-11-15 at 14 31 13

@richfab richfab changed the title Feat/external markdown Use external markdown for the spec reference Nov 15, 2023
@fredericsimard
Copy link
Contributor

LGTM !

@fredericsimard fredericsimard merged commit 2e033f9 into master Nov 15, 2023
2 checks passed
@richfab richfab deleted the feat/external-markdown branch November 16, 2023 13:47
richfab added a commit that referenced this pull request Nov 16, 2023
commit 6a49f60
Author: Fabien Richard-Allouard <[email protected]>
Date:   Thu Nov 16 11:19:08 2023 -0500

    Adds roadmap page with Productboard portal embeded (#61)

    * Adds roadmap page with Productboard portal embeded

    * Removes Draft from v3.0-RC2

    * Revert "Removes Draft from v3.0-RC2"

    This reverts commit d096251.

    * Replace portal

    * Add back the page header

commit 2e033f9
Author: Fabien Richard-Allouard <[email protected]>
Date:   Wed Nov 15 17:06:04 2023 -0500

    Use external markdown for the spec reference (#66)

    * Add mkdocs-embed-external-markdown plugin

    * Replace reference.md by the external file gbfs.md

    * Remove workflow create issue when gbfs.md changes

commit d756b22
Author: Fabien Richard-Allouard <[email protected]>
Date:   Tue Nov 14 13:16:24 2023 -0500

    Update extension tracker URL from wiki to gbfs.org (#60)

    * Adds Airtable for extensions
    * Update extension tracker URL from wiki to gbfs.org

commit 1b84777
Author: Fabien Richard-Allouard <[email protected]>
Date:   Tue Nov 14 12:28:55 2023 -0500

    Removes Draft from v3.0-RC2 (#64)
richfab added a commit that referenced this pull request Nov 16, 2023
* Adds roadmap page with Productboard portal embeded

* Removes Draft from v3.0-RC2

* Revert "Removes Draft from v3.0-RC2"

This reverts commit d096251.

* Replace portal

* Add back the page header

* Squashed commit of the following:

commit 6a49f60
Author: Fabien Richard-Allouard <[email protected]>
Date:   Thu Nov 16 11:19:08 2023 -0500

    Adds roadmap page with Productboard portal embeded (#61)

    * Adds roadmap page with Productboard portal embeded

    * Removes Draft from v3.0-RC2

    * Revert "Removes Draft from v3.0-RC2"

    This reverts commit d096251.

    * Replace portal

    * Add back the page header

commit 2e033f9
Author: Fabien Richard-Allouard <[email protected]>
Date:   Wed Nov 15 17:06:04 2023 -0500

    Use external markdown for the spec reference (#66)

    * Add mkdocs-embed-external-markdown plugin

    * Replace reference.md by the external file gbfs.md

    * Remove workflow create issue when gbfs.md changes

commit d756b22
Author: Fabien Richard-Allouard <[email protected]>
Date:   Tue Nov 14 13:16:24 2023 -0500

    Update extension tracker URL from wiki to gbfs.org (#60)

    * Adds Airtable for extensions
    * Update extension tracker URL from wiki to gbfs.org

commit 1b84777
Author: Fabien Richard-Allouard <[email protected]>
Date:   Tue Nov 14 12:28:55 2023 -0500

    Removes Draft from v3.0-RC2 (#64)

* Move roadmap to participate.md page
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