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

Cricket scorecard: Add full name when two bowlers have the same surname #27442

Open
ioannakok opened this issue Aug 28, 2024 · 1 comment
Open
Assignees
Labels
Rota Any issues that have come in as part of rota. Useful for the person on rota to track their work.
Milestone

Comments

@ioannakok
Copy link
Contributor

ioannakok commented Aug 28, 2024

Please reply to this thread when fixed: https://groups.google.com/a/guardian.co.uk/g/dotcom.platform/c/47eLqEyj6LM

When there are two players with the same surname bowling for a team, ww want to be able to tell which one of them dismissed which batter on the wickets.

See example below with two Fernandos of the Sri Lanka team.

Game: https://www.theguardian.com/sport/cricket/match/2024-08-23/england-cricket-team

Useful tool to see what data we're receiving from PA API: https://frontend.gutools.co.uk/troubleshoot/cricket

@ioannakok ioannakok added the Rota Any issues that have come in as part of rota. Useful for the person on rota to track their work. label Aug 28, 2024
@rowannekabalan rowannekabalan self-assigned this Sep 6, 2024
@ioannakok ioannakok changed the title Cricket scorecard: Add initial of name when two bowlers have the same surname Cricket scorecard: Add fullname when two bowlers have the same surname Sep 6, 2024
@ioannakok ioannakok changed the title Cricket scorecard: Add fullname when two bowlers have the same surname Cricket scorecard: Add full name when two bowlers have the same surname Sep 6, 2024
@ioannakok ioannakok self-assigned this Sep 9, 2024
@ioannakok ioannakok added this to the Requests milestone Sep 9, 2024
@ioannakok
Copy link
Contributor Author

ioannakok commented Sep 24, 2024

Here's where we've got to so far: #27464. There is a bug in this code and full name does not show as we would expect. Speak with @rtyley if you have any questions.

To test locally:

  1. We're going to use this game for testing https://www.theguardian.com/sport/cricket/match/2024-09-06/england-cricket-team because Sri Lanka has two bowlers with the same surname, Fernando.

  2. Override frontend configuration as per instructions.

  3. You need to override pa.cricket.api.key= locally. You can find the value in parameter store. Use the one for prod.

  4. Run sport app

  5. Hit localhost:9000. You need to do this first for the agents to kick off

  6. WAIT. It takes some time for the data you need to be pulled from PA to the agents.

  7. Search the logs locally. Once you see the following log message, you're ready to test:
    Image

  8. Go to localhost:9000/sport/cricket/match/2024-09-06/england-cricket-team

  9. If you want to download the real data from PA for this match in XML, you can do it using the cricket troubleshooter. The id of the match we're interested is 39145392-3f2e-8022-35f3-eac0b0654610

@CDicksonG CDicksonG modified the milestone: Requests Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rota Any issues that have come in as part of rota. Useful for the person on rota to track their work.
Projects
Status: Backlog
Development

Successfully merging a pull request may close this issue.

3 participants