Skip to content

Added help & communication section #23

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

Merged
merged 12 commits into from
Mar 24, 2019

Conversation

TomWFox
Copy link
Contributor

@TomWFox TomWFox commented Mar 16, 2019

This aims to make it more clear for people where they can get help, report bugs and join in with strategy discussions etc.

As suggested by - #5419 in the server repo.

To do:

  • Finalise links to stack overflow tags as these may be changing.
  • Change help links in the footer to reflect changes

@TomWFox TomWFox marked this pull request as ready for review March 21, 2019 01:04
@TomWFox TomWFox requested a review from acinader March 21, 2019 01:04
@TomWFox
Copy link
Contributor Author

TomWFox commented Mar 21, 2019

The footer has a link to the Parse tag on server fault - should I add that as an option in the new help & communication section?

@awgeorge
Copy link

I wouldn't let that decision hold up the progress with this ticket, we can always review it later.

While there are not many questions there, they do seem to have answers, meaning somebody is making the effort to help. I don't have the authority to make that call...

Copy link
Contributor

@acinader acinader left a comment

Choose a reason for hiding this comment

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

with one optional super nit.

@TomWFox
Copy link
Contributor Author

TomWFox commented Mar 23, 2019

I've changed the wording + links based on the conclusions from the discussion here.

I've also fixed the text overflow on .repoList .repoDescription so that it is not hidden and wraps onto more lines -
Before changes:
Screenshot 2019-03-23 at 10 05 44

After changes:
Screenshot 2019-03-23 at 10 05 21

Copy link
Contributor

@acinader acinader left a comment

Choose a reason for hiding this comment

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

nice.

@acinader acinader merged commit aef7096 into parse-community:master Mar 24, 2019
@awgeorge
Copy link

Looks good - I think it’s missing an entry in the navigation. There is a scrollspy esc menu bar.

@TomWFox
Copy link
Contributor Author

TomWFox commented Mar 24, 2019

@awgeorge good point, I’ll sort that out.

@TomWFox TomWFox deleted the help-section branch March 24, 2019 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants