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
Generate a parity list report #1775
Comments
Note: this is related to #1780 , in addition to a parity list with bvn, we must also review new features that were added to Bootstrap v5, that have not been implemented in bvn. For example, stripedColumns were an addition to bootstrap v5, but weren't added until like v0.12. These things need to be documented as well. |
I've taken a stab at creating a spreadsheet for tracking parity/completion of features. I hope the spreadsheet is self-explanatory, The first tab is definitions, and the second is the actual tracking. Design decisions, which I'm happy to take feedback on:
A read only version of the spreadsheet is here: https://1drv.ms/x/s!AiUqzkjNYGL6ieBPpQpcR41wo1laZQ?e=DYykFl - I'll share write access to contributors once we get things locked down. P.S. I posted something like this yesterday. When I looked at it again it looked like GitHub duplicated it, so I deleted the duplicate. But it appears that it disappeared. So, if the apparent duplicate was someone quoting and commenting, I apologize and hope I didn't delete too much information. Please go ahead and repost, and I promise not to delete it again…. |
It's unlikely that you'd be able to delete other persons posts. I wouldn't worry about it. Regardless, all events are sent to my email, so I can see that nobody else had made a response. So, this is a non-issue. The format looks good, this is likely how I would do it. Do keep in mind that very soon #1791 will be merged. So, some component definitions may change |
Thanks - my next step is going to be to see if I can figure out an automated way to populate the spreadsheet with the BSV Legacy information, that seems like it would give us a good framework. I think we probably want human eyes when filling in any of the BSVN information so I'm less convinced that automating that makes sense, but I'll think about that while I get the BSV Legacy step in. |
I populated the parity spreadsheet with the bootstrap vue legacy information (nearly 2000 rows) and over 100 components! I wrote automation to do this, and I believe I got everything, but could use another set of eyes to check if I missed anything obvious. I then went through and filled in the rows for BFormCheckbox and BFormCheckboxGroup since those are components that I've fixed bugs in and updated documentation for recently; note that I filled this out with the assumption that #1791 had already been done. I added a tab to the spreadsheet with instructions, which includes the process I used as a suggested process. I'm happy to refine that based on suggestions from others with more experience. Also interested if I missed anything big. Things that still need to be decided before using this to evaluate more components: Do we want to auto-populate the BSVN columns? I'm leaning towards no. I feel like the person filling out a component is going to be spending enough time reconciling the various places where items are used/defined/documented that manually adding these cells won't be much of a burden. But it also won't be too hard to generate this data - the most challenging part might be weaving the two sets of data together. I think we'll want to consider what to do about things like button-variant and models. These are things where we generally have parity but are handled a bit differently in BSVN. I would suggest that we write a general discussion of these differences in the migration guide, then when going through this spreadsheet, if that general discussion seems to cover the difference, mark the migration column "Not Needed." If that makes sense, I'll add an issue/issues for documenting each of these general issues in the migration guide (and then see if I can find time to do a first pass at them). How thorough do we want to be about documenting types? There are several types used in these components that don't show up in the types documentation; should we track that here as well? How do we want to open up access to this? I think I'd be all right with just publishing an editable link to this spreadsheet would be easiest. OneDrive manages versioning, so if a bad actor or an over-enthusiastic novice messes it up, I can just go back to a previous version. Alternatively, I can add specific users to write access. Let me know. |
Likely #1585 should be reopened. If they're used, they should likely be exported.
This is a good question with no great answer. Github does not have any builtin way to make an excel sheet, which makes sharing around difficult. Likely only myself and @xvaara need access. Although, anyone that I suppose comments here could also be given access Alternatively, it may be best to have each file be readonly, where anyone that wishes to make a change would download the sheet, make modifications, then reupload to this thread. However, I do wonder if there's security issues involved with this. Overall, very good work. I don't have any specific mentions about the structure or anything else. Migration is definitely more difficult when we have three different places of changes (bootstrap, bv, & vue). Thank you |
Do we also expect that every exported type is documented on the types page? I would say yes. In which case, maybe we can keep it simple and make verifying that part of the docs complete bar?
So it turns out that onedrive will let me create a separate read/write link that I could share privately with you and @xvaara and you could share out with others at your discretion. I could also delete that link if it leaks for some reason. Seems like a decent solution if you're both good with that. Excel Online is getting pretty good, so you probably don't have to download the file, but if only the three of us have access, it's probably not a giant security risk. Let me know your preferred channel for me to send you the link - it looks like there is a little used Discord server for this project, so I could DM you on Discord (I saw Xvaara's handle show up in the general channel, but it wasn't obvious what yours is).
Thanks! |
Good catch, @euoia . This is a general bug in the scraper that generates the initial report. I'll fix it and put a revised version up for review. More broadly - My guess from his comments on this thread is @VividLemon originally intended this to be something that he and the other maintainers (I think that might just be @xvaara at this point) could use to run through all the features before calling v1 complete. On thinking about this a bit more, my proposal is to open this up to any contributor who is willing to spend the time to dig through the code and docs to fill out the parity report in a way similar to what I did for In any case, I would appreciate some more eyes on the initial report before we open it up to editing, because it will be much harder to correct this kind of mistake once others have contributed. |
@euoia I reuploaded the parity spreadsheet with the new data which includes the "union" types. I looked through the BTable section and things appear to line up. Please let me know if you see anything else I missed. Thanks! |
All of the last comments are things I agree with. For some reason I've been forgetting about this, and didn't realize that it had gotten so old. I am okay with a general spreadsheet being passed around. It seems like the quickest way to have this issue fixed, as this is very high importance for the prospects of v1. Seems loosening restrictions is a necessity for quickness. The only reservations I have are those security related in excel itself. And you never really know with it being truly open. But it seems like a low risk high reward situation, if any risk My discord is @templeos |
I've put up a PR to update |
@VividLemon - discord wouldn't let me DM you with the password for the r/w version of the spreadsheet. I sent you a friend request, my handle is @Ohdwg, which seems like you may require to DM? |
Deprecation note #1830 (comment) |
Deprecation note #1798 (comment) |
Deprecation note |
Clear and concise description of the problem
In order to release v1, a parity list spreadsheet needs to be created for internal use. A list comparing the feature set of bv to bvn, including: props, emits, and functionality.
This is vital to the release of the library as we need to be able to properly index what is missing and what is simply removed behavior.
Even things that are declared deprecated in bv should be noted on the parity list, but have a comment describing they are deprecated.
An online spreadsheet (please do not post direct files here, for security) as an evolving spreadsheet.
Suggested solution
An online spreadsheet should be created here to document every components parity with that of bv. It should contain information about prop differences, (say if bv had type
"string"
and we had type"sm" | 'md' | 'lg'
. It's not invalid, but it needs to be documented)Alternative
This needs to be finished by the time v1 is released
Additional context
No response
Validations
The text was updated successfully, but these errors were encountered: