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

Service Rewrite Epic #351

Open
36 of 52 tasks
manio143 opened this issue Nov 9, 2022 · 0 comments
Open
36 of 52 tasks

Service Rewrite Epic #351

manio143 opened this issue Nov 9, 2022 · 0 comments

Comments

@manio143
Copy link
Member

manio143 commented Nov 9, 2022

Deadline April End of Summer 2023

Motivation

The service needs to be upgraded. It's running on a ruby stack that is no longer supported. The fronted app is served using rails webpacker, which is deprecated.

There were two choices, upgrade the app to use latest ruby and rails, or rewrite it. Unfortunately I don't feel comfortable enough with ruby to upgrade the service with confidence and fix all the breaks. I didn't want to do a rewrite but I feel it's necessary. I researched that it can still be hosted on Heroku using a docker container.

Work tracking

I'm the most familiar with .NET as that's what I use at work. ASP.NET should be a good replacement for rails, with EntityFramework Core as the database ORM.

But first I need to write tests against the existing service such that when doing the rewrite I can maintain a high degree of compatibility. This way I will also be able to highlight any hidden bugs.
See https://github.com/iqasport/referee_hub/tree/feature/dotnet-system-test
Unfortunately due to time constraints the test suite has not been implemented.

UPDATE the rewrite has been started and got to a fairly stable point so the changes got merged into the main branch where further development continues.

The following items represent work done and remaining.
1️⃣ /2️⃣/3️⃣ - priority (1 - must have, 2 - important but after release, 3 - nice to have)
👤 - user facing feature
📦 - server feature

  • Identity
    • User is able to create account, confirm email address
    • User is able to sign in with password
    • User is able to reset their password
    • User is able to change their email 2️⃣👤
  • Referee
    • Referee can modify their bio
    • Referee can set their NGB/team (bug - cannot unset)
    • Referee can see their test attempts and results
    • Referee can see available tests
    • Referee can take a test
    • Referee can pay for the HR certification
    • Referee can copy their profile URL and share it with someone else
  • NGB Admin
    • Admin can view the NGB details, player count, social media links and logo 1️⃣👤
    • Admin can edit the NGB details, player count, social media links and logo 2️⃣👤
    • Admin can invite another admin 2️⃣👤
    • Admin can import Team data 2️⃣👤
    • Admin can edit Team data 2️⃣👤
    • Admin can view their Teams 1️⃣👤
    • Admin can view stats 1️⃣👤
    • Admin can view their Referees 1️⃣👤
    • Admin can export Referee/Team data 1️⃣👤
    • Run monthly job to save NGB statistics 2️⃣📦
  • IQA Admin
    • Admin can view NGBs and make modifications 2️⃣👤
    • Admin can create a new NGB 2️⃣👤
    • Admin can import NGB data 3️⃣👤
    • Admin can view all Referees 2️⃣👤
    • Admin can create a Test 2️⃣👤
    • Admin can view and edit Test and questions 3️⃣👤
    • Admin can import/export test questions 2️⃣👤
    • Admin can view test statistics 3️⃣👤
    • Admin can grant/revoke Referee certification 2️⃣👤
    • Admin can grant access to HR payment for Referee 2️⃣👤
  • Hosting
    • New service is hosted on AWS Beanstalk
    • Service uses HTTPS to securely transport user data 1️⃣📦
    • New domain is created with Cloudflare - hub.iqasport.org - which proxies traffic to the AWS server 1️⃣📦
    • Secure cookies are enforced 3️⃣📦
    • Postgres database and Redis cache are set up on AWS and connected with the service 1️⃣📦
    • CI/CD pipeline is configured to allow triggering service updates through GitHub 2️⃣📦
    • Add Stripe public key to environment in CI build 1️⃣
    • Previous domains redirect 302 to the new one 1️⃣📦
    • OpenTelemetry integration
      • OpenTelemetry integration with AWS to track errors 3️⃣📦
      • OpenTelemetry integration with Azure to track errors 1️⃣📦
  • Data migration
    • Prepare script to perform export from postgres hosted on Heroku, import on AWS, apply initial EF migration manually 1️⃣ (Database Migration instructions #376)
    • Ensure each NGB has a country code (short name) 1️⃣
  • Testing
    • Blob storage for avatars 1️⃣👤
    • Mailing using MailerSend 1️⃣📦
    • Database 1️⃣📦
    • Stripe using test secrets with test credit card 1️⃣👤
    • Webhook Stripe integration requires testing after production deployment 1️⃣📦
  • Misc
    • Stripe webhook signature validation 2️⃣📦
    • Maintenance notifications 2️⃣👤
    • Count down rather than count up when user is taking a test 2️⃣👤
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

1 participant