Skip to content
This repository has been 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

wants to merge 2 commits into from

Conversation

devdmd
Copy link
Contributor

@devdmd devdmd commented Jun 1, 2023

@devdmd devdmd requested a review from 5A11 June 1, 2023 07:47
Comment on lines -5 to -7
<a href="/fund_form" class="announcement-link">
&gt;&gt; Apply for the $150M Developer Fund &lt;&lt;
</a>
Copy link
Member

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
Member

@5A11 5A11 Jul 6, 2023

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?

Copy link
Member

@5A11 5A11 left a comment

Choose a reason for hiding this comment

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

Just 1 comment. Also address the merge conflict and it's lgtm.

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

Successfully merging this pull request may close these issues.

2 participants