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

Remove broken technical assistance text in footer #4752

Merged
merged 1 commit into from
Dec 13, 2021

Conversation

javierm
Copy link
Member

@javierm javierm commented Dec 1, 2021

References

Objectives

Remove a text which was intended to be used with a link that has been removed since then.

@javierm javierm added the Bug label Dec 1, 2021
@javierm javierm added this to Reviewing in Consul Democracy via automation Dec 1, 2021
@javierm javierm self-assigned this Dec 1, 2021
@javierm javierm added the UX label Dec 2, 2021
@javierm javierm changed the title Remove broken technical assitance in footer Remove broken technical assistance text in footer Dec 2, 2021
@taitus taitus self-assigned this Dec 13, 2021
Consul Democracy automation moved this from Reviewing to Testing Dec 13, 2021
Originally there was a link pointing to the FAQ page but it was removed
in commit e14b7b6 because by default the FAQ page in CONSUL only
contains a placeholder text.

We aren't sure where this link should point:

* FAQ page, only if the FAQ page is published
* Help page, only when the help feature is enabled
* CONSUL technical documentation page

So, for now, we're choosing the easiest solution which is removing the
text completely.
@javierm javierm merged commit 980558c into master Dec 13, 2021
Consul Democracy automation moved this from Testing to Release 1.5.0 Dec 13, 2021
@javierm javierm deleted the remove_incomplete_text branch December 13, 2021 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

layouts.footer.contact_us has no link
2 participants