Skip to content
This repository was archived by the owner on Jul 16, 2024. It is now read-only.

fix: Removing funds form #340

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix: Removing funds form #340

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f27cf3d
fix: Removing funds form
devdmd May 31, 2023
414720d
fix: Rolling back the previous changes to keep resources
devdmd Jun 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions mkdocs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ site_description: Everything you need to know about Fetch.ai.
edit_uri: ""
site_author: developer@fetch.ai
copyright: > # Copyright notice in footer + trigger cookie consent settings
Copyright © 2018 - 2022 Fetch.ai -
Copyright © 2018 - 2023 Fetch.ai -
<a href="#__consent">Change cookie settings</a>

strict: true
Expand Down Expand Up @@ -109,7 +109,6 @@ plugins:

extra:
repo_root: https://github.com/fetchai/colearn/tree/master/
fund_form_api: https://fund-backend.fetch.ai
generator: false
analytics:
provider: google
Expand Down
7 changes: 0 additions & 7 deletions overrides/main.html
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1 @@
{% extends "base.html" %}

{% block announce %}
<!-- What you put here will be displayed as an announcement at the top of every page's header-->
<a href="/fund_form" class="announcement-link">
&gt;&gt; Apply for the $150M Developer Fund &lt;&lt;
</a>
Comment on lines -5 to -7
Copy link
Contributor

Choose a reason for hiding this comment

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

Just remove this part. Leave the other parts in case we want to add an announcement in the future.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@5A11 I have updated the pull request as suggested. Can you please take a look at the new changes?

Copy link
Contributor

@5A11 5A11 Jul 6, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

Just remove linens 5 to 7 and keep the other lines.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

{% block announce %}
 .... ....
{% endblock %}

The above block is just used to show line 5-7. If we remove line 5-7, then that code snippet does nothing. Do we still need to keep it?

{% endblock %}