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

Some (optional) changes #1

Open
gemantzu opened this issue May 18, 2019 · 1 comment
Open

Some (optional) changes #1

gemantzu opened this issue May 18, 2019 · 1 comment

Comments

@gemantzu
Copy link

  • Either remove the logo link, or make it point somewhere to the app
  • change Readme.md's last line session/new url to show just that (localhost:4000/session/new instead of localhost:4000). Also, it might be helpful to do something with root route (maybe point the user to the new session if they are not logged in, or make the chats route as the root directory.
  • Maybe merge this with master?
  • empty lib/phat_web/presence.ex and let user type the tutorial in Scratch that, I see what you did in the end.
@doomspork
Copy link
Member

@gemantzu in the interest of helping @SophieDeBenedetto get her blog out, would you be open to a PR for your suggested changes?

@idkjs idkjs mentioned this issue Nov 4, 2020
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

No branches or pull requests

2 participants