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

Suggestion on note taking #143

Open
4 of 9 tasks
joey-ma opened this issue Mar 25, 2023 · 8 comments
Open
4 of 9 tasks

Suggestion on note taking #143

joey-ma opened this issue Mar 25, 2023 · 8 comments
Assignees
Labels
complexity: missing discussion feature: stakeholder management ready for product role: product s: CTJ stakeholder: Civic Tech Jobs s: hackforla.org stakeholder: hackforla.org website s: org stakeholder: the org (includes stats) s: PD team stakeholder: People Depot Team s: VRMS stakeholder: VRMS size: 1pt Can be done in 4-6 hours

Comments

@joey-ma
Copy link
Member

joey-ma commented Mar 25, 2023

How to manage this issue

order of people

  • Fang
  • Nicole
  • Chelsey
  • Send to product by adding to our agenda

Details

Hi y'all!

With Bonnie’s help we are quickly prioritizing issues and we’re getting more clarity for a lot of issues, but I wonder if a little bit more notes (especially as it relates to understanding our requirements) could help with everyone being on the same page.

Suggestion: Not hoping to make the process more laborious, I personally am noticing a pattern and would like to just propose an idea (or 2) to help make things more clear:

Reasoning:

  • The ERD describes a complex initiative with several user stories and subtasks, and the PD: Table and field explanations is providing more details in a spreadsheet format. However, it takes quite a bit of effort (for me, at least) to remember how a table is being used in what scenarios, and oftentimes the spreadsheet also doesn’t have all the details documented. This might also be due to multiple things happening asynchronously and slowly, yet the context for each issue is often different, but it’s not being written in the comments.
  • Currently, each table name may not be descriptive enough for us to know the context as it relates to an issue, and the spreadsheet is incomplete. We can certainly remind ourselves to update the spreadsheet, or we can incorporate the following into our comments -- specifically the comments from our meetings with our customers (vrms, CTJ, etc), as we’re talking with their tech leads. We can also ask them to provide these things as well.

Suggestion:

  • User story: adding a couple of sentences to describe a user story within an issue, as we’re going through these issues could help us remember what we’re working towards.
  • Using table(s): Currently the tables in our ERD seem to be normalized, which is good for database design, but takes some effort to assume the queries or behavior expected, In a way, I might be leaning towards implementing a Backend for Frontend pattern. I believe these tables are also known as look up tables or XREF, or cross reference tables. While XREF is usually meant to increase query speeds, the purpose is much simpler: just to help with understanding. I’m not suggesting we create another database table, but am suggesting a simple spreadsheet table as part of notes and documentation. Please see examples below.

An example but using unrelated data:

We have ERD (what's on the left), but we don't have the expected data fields (bottom right, which is something similar to the table that I am thinking about, it also doesn't have to be perfect), and eventually, based on what we expect to get, the backend team can help with the actual query (upper right). Even if the frontend is to come up with the query string, knowing the expected fields and data with examples can still be helpful.

image

Example 1: (a simplified example, but maybe not the best example)

  • User story: VRMS manages users, specifically volunteers and admins.
    • On app load
      • existing users can select a meeting to check-in, or
      • check in as new user (gets the eventId from dropdown selector)
      • a new user can create new profile
  • Using table(s):
Story Scenario 1 Scenario 2 Scenario 3
State returning user new user
Action Select a meeting to check in Check in as new user Create a new profile
Options All currently available meetings (meetings that are happening right now Goes to https://www.vrms.io/checkIn/newUser?eventId=641ce82d7e68700019c01849 Goes to https://www.vrms.io/newProfile
Example 1 PM/ORG    
Example 2 ALL    
  (not sure if it’s behaving correctly)    

Example 2 (for current issue):

  • User story: When a new user signs up (on CTJ's website? or a user can update their info on their profile at anytime on VRMS, which the same data will be used by CTJ's website?), user will fill out his/her/their user profile. We'd expect them to fill out something like the following table.

image

  • Questions to think about:
    • Do we need to provide them with every Job_Function category? examples: Design, Research, Engineering, Product... etc
    • Is there a better word for this data? Job_Field or Job_Department or Functional_Area or "Competencies"?
    • Is the role table referring to "Software Engineer" (title) or "Front-end Development" (position / area of responsibility), or "Tech Lead" (leadership)? Note leadership_type and permission_type and role are all each a table.

Here is my current understanding:

I'm imagining: "Fang" who has the role of a "Software Engineer", with the leadership_type of "Tech Lead" for project "PeopleDepot" with permission_type "PeopleDepot Admin" will go to "somewhere_page" to enter user.name_first and user.name_last "Joey Ma" (which will query from all list of users, or all users within project "PeopleDepot", and upon selecting this user (by user.id), Fang can then change my leadership_type to "Interim Tech Lead" and permission_type "PeopleDepot Admin"...

However, this is more of a guess than an actual understanding of the requirements. Having an actual example would be nice. At this time, I'm actually not quite sure how an "admin user" will be able to assign permission to another user. Someone else could have a different version of what the user is expected to see, select, and enter.

While this user story is not considering quite everything, having some written examples of user stories will help with remembering what the devs are developing for and thereby how to optimize code and db design. The dev team will eventually need some example data to appropriately come up with the each query, and clear up what the best data type to store the data would be.

some table that helps to illustrate the the relationship

Originally posted by @yoyoyojoe in #142 (comment)

@joey-ma joey-ma changed the title Hi y'all! Suggestion on note taking Mar 25, 2023
@ExperimentsInHonesty ExperimentsInHonesty added discussion role: dev lead size: 1pt Can be done in 4-6 hours s: CTJ stakeholder: Civic Tech Jobs s: VRMS stakeholder: VRMS s: hackforla.org stakeholder: hackforla.org website s: PD team stakeholder: People Depot Team s: org stakeholder: the org (includes stats) feature: stakeholder management labels Apr 6, 2023
@ExperimentsInHonesty ExperimentsInHonesty added this to the 3 - Project Mgmt milestone Apr 6, 2023
@fyliu
Copy link
Member

fyliu commented Apr 7, 2023

Should we have a data catalog?

At the meeting, Chelsey suggested recording this metadata in a data catalog, which is a tool intended to be used by data professionals (data engineers, data scientists, data stewards, and chief data officers). It doesn't directly have a benefit for the software engineering side of the project, but if it's not too much work, I think we should attempt to fill in this document in the spirit of striving to be a better project.

Where to store the data catalog?

I was personally thinking to make note of things in maybe the create table issues, which helps guide the developer when they work on them, and then the developer should copy or write a more appropriate version of it as documentation that becomes part of the PR along with the code (Documentation as code). I don't know if it makes sense for the data catalog to be part of the documentation that's in the code repository, but it would make it easier for the developer to work on it since it's accessible in VS Code just like the documentation and the source code. The developer would just need to do an extra copy/paste to the data catalog file without leaving VS Code or whatever editor they use. I'm certain we will have search functionality in whatever documentation system we choose to use.

On the other hand, having a standalone document might be easier for a data person to consume.

@fyliu
Copy link
Member

fyliu commented Apr 7, 2023

Comments

  1. We need more context for the table issues, to note the scenarios of how the table is being used.

    Comment: I agree, and I assume this information is for the benefit of the developer working on the issue of implementing the model?

  2. Some table names don't precisely reveal their intended use, like role.

    Comment: It's good to point out these things and I think the solution is to rename them to be more specific whenever possible.

  3. We need example data for the models.

    Comment: I can see how this can be converted into unit tests. Having agreement on which fields are nullable/blankable helps. Currently, for me, I assume most fields can be empty (e.g. optional). The reason is that a client using the API would be able to use it without including all the fields. This is actually not great design. One of the dangers is that we're providing a data field that no client actually use or need, and we don't know that we could remove it. The better way (best practice) is to implement only the fields that some client asks for (will use). This is something we need to discuss and decide on as a team. My original assumption was based on the thought that "VRMS UI don't be caught up to be able to provide all the model's data when they integrate".

  4. Example 1 user stories

    Comment: I assume this benefits the developer. But it feels like something that's not readily consumable for the developer. Further developing them into use cases might be a better representation format.

    For Scenario 1, I think a more useful representation would be this:

    • Use case: A logged in user needs to be able to retrieve a list of all currently available meetings (meetings that are happening right now) to check in to. (this makes it clearer that we're talking about user and event models, and leaves out the UI elements like dropdown selector and app load)
    • Use case: A logged in user should be able to check in to a meeting
    • Example values: PM/ORG, ALL

    Scenario 2 might be something like this:

    • Use case: A guest should be able to retrieve a list of all currently available meetings
    • Use case: A guest should be able to create a new user and check in to a meeting at the same time

    Scenario 3 could be like this:

    • Use case: A guest should be able to create a new profile

    Comment: So this can become 5 use cases with 2 actors (guest, logged in user). These can translate directly into acceptance criteria for use in tests.

  5. Example 2

    It's not mentioned in the user story, but I assume this is about the role table.

    Types of questions:

    • question about the extent of the pre-populated data requirement
    • question about naming of the data, with suggestions
    • question about the nature of the data being stored in the table, with examples, when the table name doesn't allow one to figure it out

    Comment: I think its great to ask questions and work out a reasonable example to draw out any unwritten requirements or come to a better/clearer design.
    I think all the questions should be made into task items for discussion. Then we can check them off when we're done with them. Note any decisions that were made or new issues stemming from them.

@fyliu fyliu assigned Neecolaa and unassigned fyliu Apr 7, 2023
@joey-ma
Copy link
Member Author

joey-ma commented Apr 8, 2023

  1. We need more context for the table issues, to note the scenarios of how the table is being used.
    Comment: I agree, and I assume this information is for the benefit of the developer working on the issue of implementing the model?

Definitely. A descriptive prompt with different examples help developers better understand the problem and therefore can focus more energy on coming up with a solution. In our case, we have a general understanding of our goal, but it'd be hard to be caught up on all previous discussions among a cross-functional team. I definitely understand it's an iterative process and we are getting closer and closer to our goal as we go, but writing down the context and examples for the table issues helps to document the requirements, even if the note itself is improved incrementally.

  1. Example 1 user stories
    Comment: ...

We can start somewhere and improve it incrementally as requirements become more and more clear. Certainly more examples / scenarios help developers come up with the solution that meets the expectation(s).

I think all the questions should be made into task items for discussion. Then we can check them off when we're done with them. Note any decisions that were made or new issues stemming from them.

💯. And having "user stories" and "tables" (that help illustrate scenarios or expected data) are meant to help convert clarifying questions into actionable steps. 🙏🏻

@Neecolaa
Copy link
Member

Neecolaa commented Apr 9, 2023

About who writes the user stories:

I'd think the team implementing the front end would write the user stories. Our direct users would be the teams using the API (or APIs if we do a Backend For Frontend), as we don't have our own front end outside of that.

When we have an idea of how something will work, we can write the user stories and ask for verification from the given group. So for the check-in example, VRMS would be the ones either creating the user stories, or verifying that the stories we wrote were correct.

I did find the user story table a little difficult to read. Are the examples listed for Scenario 1 examples of meeting titles/types?

About who writes the queries:

I'd think we'd be the ones who'd write the queries based off what stakeholders want (like what's in #2).

Would writing views be helpful here? Really they're just stored queries.
Implementing views in django

About expected data:

The data dictionary Chelsey started will be great for this. The fields tab in the PD: Table and field explanations has some information we can transfer to the data dictionary, but it's incomplete.

For tables where we have the data already, I have added link symbols to the right corner of the table's header. Clicking that will open up the data tab for that table in the PD spreadsheet. Not as convenient as the data dictionary will be, but still something for now.
Program area table on ERD with link symbol clicked
Corresponding Program Area data tab in PD spreadsheet

Question on XREF tables:

Which is the example for the XREF table? I'm most familiar with xref tables being like this, but I don't think that's what you mean here.

If xref refers to some other common idea, I have a potential concern:
Currently, we use xref in our table names whenever we have a table implementing a many-to-many relationship (ex: project_language_xrefstores the relationships between many projects and the many languages they use). Should we change this? Initially, we used "join" (ex: project_language_join), but changed it to avoid the JOIN keyword. This example doesn't use a keyword, but I like having one so we can find these types of tables easier. Maybe "link"?

@Neecolaa Neecolaa assigned chelseybeck and unassigned Neecolaa Apr 9, 2023
@fyliu
Copy link
Member

fyliu commented Apr 9, 2023

I started to write something more and it turned out really long. I'll shorten it here by leaving out the details.

The API abstraction is there for a reason. The backend mentality should be to work towards the API.

It's a mistake to work on user stories backwards from tables in the backend. User stories is on the opposite side of the API from the backend.

The frontend needs to develop and keep all the user stories and work with the backend to come up with an API that satisfies the data usage requirements.

Just having the table definitions like we have isn't an API. We need to establish actual endpoints and a set of behaviors as acceptance criteria.

This makes me want to recommend a break from our current workflow and work first on specifying the API with the frontend team, which is VRMS right now.

User stories are not appropriate to include in table issues for other reasons.

@ExperimentsInHonesty
Copy link
Member

@yoyoyojoe lets chat briefly about this tomorrow....

Summary is we have ways of doing what you are asking for, the blocker here is my and nicole's time.

@shmonks
Copy link
Member

shmonks commented Oct 9, 2024

@shmonks shmonks self-assigned this Oct 10, 2024
@shmonks

This comment has been minimized.

@shmonks shmonks moved this from ❓Questions/Review to 🏗In progress-actively working in P: PD: Project Board Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: missing discussion feature: stakeholder management ready for product role: product s: CTJ stakeholder: Civic Tech Jobs s: hackforla.org stakeholder: hackforla.org website s: org stakeholder: the org (includes stats) s: PD team stakeholder: People Depot Team s: VRMS stakeholder: VRMS size: 1pt Can be done in 4-6 hours
Projects
Status: 🏗In progress-actively working
Development

No branches or pull requests

6 participants