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

feat: create line profile page #216

Merged
merged 27 commits into from
Dec 2, 2023
Merged

feat: create line profile page #216

merged 27 commits into from
Dec 2, 2023

Conversation

VSciFlight
Copy link
Contributor

Description

Added a profile page accessible only by the path of '/profile'

  • it has the inputs to determine the line the user intends to see
  • an empty timetable
  • presumably a timeline graph of that bus comparing the schedule against reality (now it is recommended to see it as template).

things left to do:

  • integrate the inputs to the profile so that the profile shows the info according to the line it received
  • fixed timetable that fetch the timetable for that line
  • graph shows relevant information

screenshots

image

VSciFlight and others added 8 commits October 21, 2023 12:45


* 'main' of https://github.com/VSciFlight/open-bus-map-search:
  Added Unit Test for Menu.tsx (#154)
  Added Localization support for About and Dashboard (#95)
  docs: update link to api repo at readme (#150)
  updated HbarChart style (#152)
  Reverse the timeline graph XAxis to be with  'ltr' orientation (#149)
  remove unnecessary console.log
  remove unnecessary console.log
  hotfix: operatorRefField (#142)
  [Bug] Fix GapsPatternsPage layout
  [Bug] Fix Dashboard Layout (also in Mobile)
  docs: better styling for testing instructions
  Test - use advanced HAR (#140)
  Add github lint workflow file (#141)
  Refactoring of the Sidebar component and its styling (#138)
  feat(useVehicleLocations): support operatorRef (#96)
  feat: auto complaint generation (#136)
  Create dependabot.yml (#135)
  Update package.json
  refactor: test commands
changed directory for the gaps component
Copy link

netlify bot commented Nov 14, 2023

Deploy Preview for astonishing-pothos-5f81bd ready!

Name Link
🔨 Latest commit 1dbd72a
🔍 Latest deploy log https://app.netlify.com/sites/astonishing-pothos-5f81bd/deploys/655d2fcf8c08e700086dcd9a
😎 Deploy Preview https://deploy-preview-216--astonishing-pothos-5f81bd.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

src/pages/Profile.tsx Outdated Show resolved Hide resolved
Copy link
Member

@NoamGaash NoamGaash left a comment

Choose a reason for hiding this comment

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

Good work 🙏

src/pages/components/BusArrivalsTimeline.tsx Outdated Show resolved Hide resolved
src/pages/components/BusArrivalsTimeline.tsx Outdated Show resolved Hide resolved
src/pages/components/BusArrivalsTimeline.tsx Outdated Show resolved Hide resolved
})
{
return (
<div className="chart">
Copy link
Member

Choose a reason for hiding this comment

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

please split to smaller sub components

VSciFlight and others added 6 commits November 15, 2023 11:10
…us-map-search into profile_page_1

* 'profile_page_1' of https://github.com/VSciFlight/open-bus-map-search:
  chore: upgrade linter (#217)
  test: improve dashboard and single-line tests (#192)
  ci: publish storybook (#213)
…us-map-search into profile_page_1

* 'profile_page_1' of https://github.com/VSciFlight/open-bus-map-search:
  Slack link update (#225)
  issue #205: GitHub repo link (#221)
  #issue 187 lazy load - dynamic import (#218)
src/pages/Profile.tsx Outdated Show resolved Hide resolved
@shootermv
Copy link
Collaborator

i need some explanation please, if i want to access the profile page, where should be a link leading to it?

@VSciFlight
Copy link
Contributor Author

@shootermv there isn't a link at this point due work in progress. you can access the profile page at "/profile"
image

@NoamGaash
Copy link
Member

@VSciFlight have you seen the linter output?
thanks

@NoamGaash
Copy link
Member

image
you can also see it inside github

@NoamGaash NoamGaash linked an issue Nov 21, 2023 that may be closed by this pull request
@NoamGaash NoamGaash changed the title issue #193: Profile page creation and a possible template feat: create line profile page Nov 21, 2023
@VSciFlight
Copy link
Contributor Author

So, I have made changes to the page and left the modules that don't crash (for now) or having issues with them.

@NoamGaash NoamGaash merged commit 64e43ea into hasadna:main Dec 2, 2023
9 checks passed
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

Successfully merging this pull request may close these issues.

Profile page for lines
3 participants