Comments (5)
Did you mean 'weekly bosses talent materials'? If so, I think I found the solution for it.
I took the Dvalin's Plume image from the wiki.
Here's an example with Fischl:
Technically, I made it so the weekly bosses materials are taken from the second element of the talentMaterials
array in the characters.ts
file, as we know that one character only needs one weekly ascension material. I also made it so that Trounce
domains could be done AnyDay
. If we want to be perfectionist, we can add the Crown of Sagehood too haha!
We also have to modify the domainDropSets.ts
and talentMaterials.ts
to add each of the weekly bosses material to each character.
What do you think of this? (I know about the double checkbox thingy, will probably have to do a workaround of it, maybe by using two checkboxes passing different props to the Toggle.tsx
component?)
from genshin-schedule.
I made it so the weekly bosses materials are taken from the second element of the talentMaterials array in the characters.ts file
I think it's better to go with a separate field like talentMaterialWeekly
because the main character actually requires different talent materials at different levels (see #11).
If we want to be perfectionist, we can add the Crown of Sagehood too haha!
Crown isn't farmable from domains so unfortunately it doesn't belong in the Domain List.
I know about the double checkbox thingy, will probably have to do a workaround of it, maybe by using two checkboxes passing different props to the Toggle.tsx component?
I'm not too worried about code duplication at this stage.
Otherwise I think it's great! If you have the code for this already, I think you should make a draft PR and we can work together from there.
from genshin-schedule.
I think it's better to go with a separate field like talentMaterialWeekly because the main character actually requires different talent materials at different levels (see #11).
Looking at the wiki, I believe the Traveler only needs a Dvalin's Sigh
for weekly talent materials. But sure! I also think that it is better to split them up for modularity, and because we never know when the developers might change the weekly materials (imagine requiring multiple weekly materials for a single talent level up ๐ค).
Otherwise I think it's great! If you have the code for this already, I think you should make a draft PR and we can work together from there.
Alright! Let me adjust the code to your request for a bit before I make the draft PR!
EDIT: Before I forget, I think it's also good if we add the elite bosses for the ascension materials (hoarfrost cores, basalt pillars, everflame seeds, etcetera). What do you think? I have no idea where to place them in the domains, though.
from genshin-schedule.
I believe the Traveler only needs a Dvalin's Sigh for weekly talent materials
I'm talking about this row on the Traveler page, which shows that different talent materials are required at different levels
in contrast to other characters e.g. Zhongli who only requires one type.
The array for talentMaterials
is to be used to eventually implement #11, which is why you can't designate the second array element for the weekly material.
I think it's also good if we add the elite bosses for the ascension materials (hoarfrost cores, basalt pillars, everflame seeds, etcetera). What do you think? I have no idea where to place them in the domains, though.
To be honest, I was against this idea in the beginning because I wanted the page to be strictly focused on "domains" only, but I'm convinced that it would be much more useful to include those items as well. It should probably come as a separate PR though.
from genshin-schedule.
Okay, I understand what you're talking about. No worries there.
To be honest, I was against this idea in the beginning because I wanted the page to be strictly focused on "domains" only, but I'm convinced that it would be much more useful to include those items as well. It should probably come as a separate PR though.
Sure! If I have any idea about it, I'll make an issue and a PR too. It would have to be after this PR, though ๐ .
By the way, the draft PR is already set up and ready to be worked on together!
from genshin-schedule.
Related Issues (20)
- Merging of `sync` project into `web`
- Show timestamp of when resin and realm currency became full HOT 1
- Can I logon with github accounts๏ผ HOT 1
- Outdated map HOT 1
- Farming helper / counter HOT 9
- Support for custom timezones
- Show realm currency steps and sync with the resin estimator HOT 11
- i18n
- Multiple resin notifications and estimated time in notification messages
- Use `Accept-Languages` header to determine default language
- Marking tasks as done using the Discord bot
- Allow crossing off trounce domains for the week HOT 5
- Traveller is not available as a character
- Use `scrollRestoration` next.js flag
- +60 resin button for Transient/Fragile Resin HOT 5
- List of incorrect/missing database entries
- Sometimes Logout button refuses to work
- Missing Song of Broken Pines
- Resource respawn time algorithm needs to be updated
- 2.1 CHARACTERS
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
๐ Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. ๐๐๐
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google โค๏ธ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from genshin-schedule.